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

CommentFileSizeAuthor
#5 3160308-5.patch953 bytesnarendra.rajwar27
#2 3160308-2.patch355 byteskarishmaamin

Comments

karishmaamin created an issue. See original summary.

karishmaamin’s picture

karishmaamin commented
Status: Active » Needs review
StatusFileSize
new 3160308-2.patch355 bytes
mrinalini9’s picture

mrinalini9 commented
Status: Needs review » Needs work

Reviewed patch #2 and found that core_version_requirement: ^8 || ^9 needs to be added in it's submodule's info.yml file as well i.e., modules/eloqua_api_auth_fallback/eloqua_api_auth_fallback.info.yml. So, moving this to Needs work.
Thanks!

narendra.rajwar27’s picture

narendra.rajwar27
he/him
Primary language Hindi
Location India
commented

Working on this issue. will update patch shortly.

narendra.rajwar27’s picture

narendra.rajwar27
he/him
Primary language Hindi
Location India
commented
Status: Needs work » Needs review
StatusFileSize
new 3160308-5.patch953 bytes

Updating patch as per comment #3.

Thanks

fernando.magan’s picture

fernando.magan commented

Do you plan to apply this patch to the development or stable version? We would like to be able to use this module in Drupal 9 from composer. I could help this to be fixed if necessary.

papagrande’s picture

papagrande
Primary language English
commented

Any update on getting this patch committed and new version released?

papagrande’s picture

papagrande
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

We are using this patch successfully on a Drupal 9.x site.

papagrande’s picture

papagrande
Primary language English
commented
Version: 8.x-1.x-dev » 2.0.x-dev
Status: Reviewed & tested by the community » Fixed

Thanks, @narendra.rajwar27

Status: Fixed » Closed (fixed)

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