Problem/Motivation

The module is not compatible with Drupal 10.

With a new minor, we can also remove the migration path test.

CommentFileSizeAuthor
#5 3358211-5.patch1.15 KBflocondetoile
#2 request_data_conditions-drupal_10_update-3358211-1.patch1.08 KBisidro.garcia.garrido

Comments

isidro.garcia.garrido created an issue. See original summary.

isidro.garcia.garrido’s picture

isidro.garcia.garrido commented
StatusFileSize
new request_data_conditions-drupal_10_update-3358211-1.patch1.08 KB

Adding patch to make the module D10 Compatible.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

Can we please get this committed? This was one module blocking us from upgrading to Drupal 10.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Active » Reviewed & tested by the community
flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
StatusFileSize
new 3358211-5.patch1.15 KB

Looks like findConfigEntityDependentsAsEntities() has been deprecated in 9.3, So I guess the new method findConfigEntityDependenciesAsEntities() has been introduced with 9.3. So upgrade the requirement to ^9.3.

Also dropping support of Drupal 8.

flocondetoile’s picture

flocondetoile
Primary language French
Location Lyon
commented
Status: Reviewed & tested by the community » Fixed

Just published a new release : 8.x-1.0-beta2. I guess we could publish a stable release within one month if there is still not issue reported with this module.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

Thanks so much for properly testing and adding the additional fixes, and cutting a release! 🙏

Status: Fixed » Closed (fixed)

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