- Issue created by @roaldnel
- Status changed to Needs review
over 1 year ago 10:01am 11 October 2023 - 🇳🇱Netherlands roaldnel
The patch was updated with codesniffer improvements applied.
- Status changed to Needs work
over 1 year ago 8:32am 24 November 2023 - 🇳🇱Netherlands arantxio Dordrecht
+++ b/cm_commerce.info.yml @@ -3,6 +3,6 @@ type: module +core_version_requirement: ^8 || ^9 || ^10
We add D10 support here, but that is handled in another issue so it doesn't fit in here. 📌 Drupal 10 update Needs work
Otherwise I do not see any problems here.
- Status changed to Needs review
over 1 year ago 8:49am 24 November 2023 - 🇳🇱Netherlands roaldnel
Thanks Arantxio, I have removed the D10 support change and I also removed support for Drupal 8 as you suggested in this issue: https://www.drupal.org/project/cm_commerce/issues/3386410#comment-15332689 📌 Drupal 10 update Needs work .
- 🇳🇱Netherlands roaldnel
Added D10 support back in since it has been released already, the updated patch is attached.
- Status changed to RTBC
over 1 year ago 8:55am 24 November 2023 - 🇳🇱Netherlands arantxio Dordrecht
Thanks for the updated code @roaldnel, looks fine to be released.
-
roaldnel →
committed 729ba220 on main
Issue #3393194 by roaldnel: Updated processing and error handling
-
roaldnel →
committed 729ba220 on main
- Status changed to Fixed
over 1 year ago 10:23am 24 November 2023 - 🇳🇱Netherlands roaldnel
The code has been merged and Drupal 8 support was dropped.
Automatically closed - issue fixed for 2 weeks with no activity.