- Issue created by @Liam Morland
- 🇨🇦Canada joseph.olstad
This should actually go into the 6.0.x branch which hasn't yet been created.
- 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
This change doesn't have to wait until Drupal 11. It can happen anytime that the minimum is Drupal 10.2
- 🇨🇦Canada smulvih2 Canada 🍁
This makes sense to include in the 5.2.x branch and the new 5.3.x branch (still in testing).
- 🇨🇦Canada joseph.olstad
From my searches of our code base, this change is appears to be quite extensive and non-trivial.
Perhaps target this for 5.3.x instead of 5.2.x with option of deferring to 6.0.x
- 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
Actually, this is not a change for Drupal 11. The change happened in Drupal 10.2. It looks like the setting didn't work so they just removed it. The setting can probably just be removed from the Views. The change was in commit f890b7f.
- 🇨🇦Canada joseph.olstad
ah ok, I'm not aware of anything terribly broken in 5.2.x relating to this however ya, it seems like I've seen reminents of this in config export diffs.
- 🇨🇦Canada joseph.olstad
Ok, if it's just this, then it's trivial.
There may be more to this however. - Assigned to smulvih2
- Status changed to Needs review
4 months ago 4:40pm 5 September 2024 - 🇨🇦Canada joseph.olstad
Ya, I had another look at the change notice, this is something trivial, just some cleanup for a D7 views attribute that was never working in D8+ views.
Related change notice →
see patch 8 above - Assigned to joseph.olstad
- Assigned to smulvih2
-
smulvih2 →
committed 591894c4 on 5.3.x
Issue #3472124 by joseph.olstad, liam morland: Remove...
-
smulvih2 →
committed 591894c4 on 5.3.x
- 🇨🇦Canada smulvih2 Canada 🍁
@liam good catch on this deprecation, and thanks for the patch @joseph! This has been merged with 5.3.x
Automatically closed - issue fixed for 2 weeks with no activity.