Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Add core_version_requirement: ^8 || ^9 to message_notify.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.
Comment | File | Size | Author |
---|---|---|---|
#4 | 3120194-4.patch | 1.34 KB | heddn |
| |||
#4 | interdiff_2-4.txt | 1.03 KB | heddn |
Comments
Comment #2
mo_farhaz CreditAttribution: mo_farhaz at Specbee commentedAdded core version requirement.
please review it
Comment #3
Suresh Prabhu Parkala CreditAttribution: Suresh Prabhu Parkala at Specbee commentedThanks @mo_farhaz
Patch applied without error.
Comment #4
heddnTested module w/ private message on 9.1.x. Notification emails are received by users. But to install a Drupal 9 site, I needed to update .info.yml files for the test and example modules as well. See updated patch.
Comment #5
pratik_kambleComment #6
harika gujjula CreditAttribution: harika gujjula at Specbee commentedComment #7
harika gujjula CreditAttribution: harika gujjula at Specbee for Drupal India Association commented@heddn Patch Applied successfully. But Since, Message notify has a dependency on Message module, which doesnot have the D9 compatibile version, it restricts from installing the module.
However, applied the available patch https://www.drupal.org/project/message/issues/3113123 for Message and then the module seems to work fine.
Comment #8
harika gujjula CreditAttribution: harika gujjula at Specbee for Drupal India Association commentedComment #9
hexblot CreditAttribution: hexblot as a volunteer commentedMessage core has had a D9 release for a couple of weeks now -- maybe push this change?
Seems to be working for us as well, would love to get rid of the (semi-)manual patching.
Comment #11
jhedstromThanks all!