- Issue created by @yaqbick
- 🇨🇦Canada joelpittet Vancouver
@yaqbick Could you make this a merge request please? Also before you do that can you have a peek at the work done here: 📌 Support Drupal >=10.3. Modernize the code Active
- First commit to issue fork.
- 🇨🇦Canada joelpittet Vancouver
This really looks like a duplicate of what is in 📌 Support Drupal >=10.3. Modernize the code Active , am I miss reading or should it be closed as a duplicate?
- 🇷🇴Romania claudiu.cristea Arad 🇷🇴
This really looks like a duplicate...
That's true but I would go with this to lower the footprint of the other.
-
claudiu.cristea →
committed 23f9a893 on 8.x-1.x authored by
yaqbick →
Issue #3511201 by yaqbick, alorenc, joelpittet, claudiu.cristea: Add OG...
-
claudiu.cristea →
committed 23f9a893 on 8.x-1.x authored by
yaqbick →
- 🇷🇴Romania claudiu.cristea Arad 🇷🇴
This is straight refactoring. And looks OK. Merged
PS: I will adapt the other MR
- 🇵🇱Poland alorenc Wolsztyn, 🇵🇱
I can see a potential issue here: OG 8.x-1.x can be installed on D9, but attributes were added in D10.
- 🇷🇴Romania claudiu.cristea Arad 🇷🇴
You're right. I will revert the merge and issue a new release In 📌 Support Drupal >=10.3. Modernize the code Active we will only support >=10.3 and that will go in 2.x. Let's keep the work there
-
claudiu.cristea →
committed 5330dd80 on 8.x-1.x
Revert "Issue #3511201 by yaqbick, alorenc, joelpittet, claudiu.cristea...
-
claudiu.cristea →
committed 5330dd80 on 8.x-1.x