Looks like the module has no deprecated code usages, so we just need to mark it as compatible with D9.

So, add core_version_requirement: ^8 || ^9 to info.yml file, and also add the composer.json definition per https://www.drupal.org/node/3070687.

CommentFileSizeAuthor
#4 3101523-4.patch399 bytesjcnventura
#4 interdiff_2_4.txt349 bytesjcnventura
#2 3101523-2.patch400 bytesSahana _N
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sahana _N created an issue. See original summary.

Sahana _N’s picture

Sahana _N CreditAttribution: Sahana _N commented
Status: Active » Needs review
FileSize
3101523-2.patch400 bytes

Please review the patch.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Needs review » Needs work

Module will still be compatible with older versions of Drupal 8. Please leave the core: 8.x in place.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Needs work » Needs review
FileSize
interdiff_2_4.txt349 bytes
3101523-4.patch399 bytes

Adding back compatibility with versions of Drupal older than 8.7.7.

jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Issue tags: +Drupal 9 compatibility
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Status: Needs review » Reviewed & tested by the community

Nice updates. LGTM.

Then, could we tag a new tagged release so site owners do not need to depend on a dev branch to get D9 support. Also, populate this project's D9 Compatibility field on the project's project page showing status for D9.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Consensus Enterprises for Health Canada - HPFB commented

Marked #3149279: Automated Drupal 9 compatibility fixes as a duplicate of this issue.

  • jcnventura authored 2c3489a on 8.x-1.x 
    Issue #3101523 by jcnventura, Sahana _N, heddn: Mark as compatible with...
jcnventura’s picture

jcnventura CreditAttribution: jcnventura at 1xINTERNET commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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