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

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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Sahana _N’s picture

Status: Active » Needs review
FileSize
501 bytes

I updated the info.yml please review the patch.

idebr’s picture

Status: Needs review » Needs work

Hi Sahana _N,

The instructions also mention the project has to update its composer.json file:

Please note you also need to add the same requirements from your info.yaml file in your composer.json file:

{
  "require": {
    "drupal/core": "^8.7|^9.0"
  }
}

See https://www.drupal.org/node/3070687

Sahana _N’s picture

Status: Needs work » Needs review
FileSize
905 bytes
358 bytes

Thanks for the review, I updated the patch, Please review.

  • Berdir committed 4a55b92 on 8.x-3.x authored by Sahana _N
    Issue #3097426 by Sahana _N: Mark as compatible with Drupal 9
    
Berdir’s picture

Status: Needs review » Fixed

Thanks.

  • Berdir committed 9ff903d on 8.x-3.x authored by Sahana _N
    Issue #3097426 by Berdir: Update remaining references to entity manager...
Berdir’s picture

Pushed a follow-up commit, realized there we were still mocking the EntityManager in the unit test.

Status: Fixed » Closed (fixed)

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