- Issue created by @svendecabooter
- @svendecabooter opened merge request.
- Status changed to Needs review
about 2 years ago 8:09am 22 February 2023 - 🇳🇱Netherlands ndf Amsterdam
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? - 🇧🇪Belgium svendecabooter Gent
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. - Status changed to Fixed
about 2 years ago 2:59pm 28 February 2023 - 🇳🇱Netherlands ndf Amsterdam
Thanks!
It's in this new 2.10 release [#2261853]
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
almost 2 years ago 1:34pm 7 April 2023 - 🇧🇪Belgium stijnhau
Its is kinda compatible.
The test (sub_)module isn't