We are using this module on a Drupal 9 project, that we wish to upgrade to Drupal 10.
It seems there is no support for Drupal 10 yet though.
Is this module still actively being maintained? If not, is it an option to add me as a co-maintainer, so I can keep it (minimally) going?

Issue fork commerce_mollie-3343626

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

svendecabooter created an issue. See original summary.

svendecabooter opened merge request !4

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Status: Active » Needs review
ndf’s picture

ndf commented

Hi svendecabooter, thanks you want to help maintaining this module. I have just given you the maintainer permissions.

Regarding this patch, I have one question regarding the change in the getNotifyUrl method.
Can you please check that?

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

Thanks for adding me as a co-maintainer. I'll look into creating a new release soon.

Regarding your question (also commented on the MR):

This is changed based on a deprecation notice for Commerce:
https://git.drupalcode.org/project/commerce/-/blob/8.x-2.x/modules/payme...
However this is not strictly related to Drupal 10 support.
Since composer.json for this module specifies "drupal/commerce": "~2.20", I think this should be safe.

  • ndf committed b402b5fb on 8.x-1.x 
    #3343626 by svendecabooter: Drupal 10 support
ndf’s picture

ndf commented
Status: Needs review » Fixed
ndf’s picture

ndf commented

Thanks!

It's in this new 2.10 release https://www.drupal.org/project/commerce_mollie/releases/8.x-1.10

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

Great, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

stijnhau’s picture

stijnhau commented

Its is kinda compatible.
The test (sub_)module isn't