- 🇸🇦Saudi Arabia samaphp Riyadh, SA 🇸🇦
Thank you guys, we hope to see this merged soon. thanks
- First commit to issue fork.
- last update
12 months ago 4 pass - @ankondrat4 opened merge request.
- last update
12 months ago 4 pass - 🇺🇦Ukraine ankondrat4
Hello.
+1 RTBC, #3 works on 9.5.9Besides, I have created issue fork and MR with these changes for next release of module.
- 🇫🇷France O'Briat Nantes
I add it to the global issue #3377377 about drupal-check
- Status changed to Needs work
9 months ago 12:12am 27 September 2023 - 🇺🇸United States TR Cascadia
Remove the function. In the feature, we can always reintroduce it.
I strongly disagree. It should be ported, NOT deleted, because if it deleted it will never be reintroduced. That's what has happened with many other things.
It is really no harder to port it than it is to remove it. This is what the parent META issue is all about.
Throwing away things and taking quick fix shortcuts is one of the reasons this module has deteriorated since D7. If it's important to you, make the small effort to port it properly, don't just throw away what was once working and tested code.
- heddn Nicaragua
See comments in #3287600-88: Drupal 10 compatibility fixes → for what work is remaining here. Basically, we need to figure out if the features are replaced with other theme templates or convert it. I couldn't find where it is used currently, but that doesn't mean it isn't in use. But it doesn't block D10 support.
- 🇯🇴Jordan Anas_maw
Reroll patch in #3 to work on the latest dev version for temporary solution