Add core_version_requirement: ^8 || ^9 to name.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

CommentFileSizeAuthor
#2 name-3128599-2.patch323 byteskarishmaamin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

karishmaamin created an issue. See original summary.

karishmaamin’s picture

karishmaamin CreditAttribution: karishmaamin commented
Status: Active » Needs review
FileSize
name-3128599-2.patch323 bytes

please review

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
CreditAttribution: nkoporec as a volunteer and at Agiledrop - Your Trusted Drupal Teammates commented
Status: Needs review » Needs work

We first need to fix the Drupal 9 compatibility issues, listed in here: https://www.drupal.org/project/name/issues/3128409, before merging this.

Alan D. credited Farnoosh.

Alan D. credited MegaChriz.

Alan D. credited RuslanP.

Alan D. credited joshua.boltz.

Alan D. credited wrd.

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented

Adding some additional users that helped with the main D9 patch (apologises if I missed anyone)

  • Alan D. committed 6f8e89a on 8.x-1.x authored by karishmaamin 
    Issue #3128599 by karishmaamin, Farnoosh, nkoporec, MegaChriz, RuslanP,...
Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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