- Merge request !102Issue #3265646: Group content form irregularities due to content form check for group content → (Merged) created by jeni_dc
- 🇨🇦Canada gwvoigt London, ON 🇨🇦
Not sure if it's related since I'm not using Group Term or Groupt Term Menu.
I'm using the patch from #5 and still getting an inconsitency when comparing the node add form to the group node add form:
The sidebar does not render correctly, it renders at the bottom of the form, and the toggle doesn't work either
- 🇨🇦Canada gwvoigt London, ON 🇨🇦
Group 3.x has different naming pattern in its machine names for 2 entity types so I added a condition to account for the new version as well when checking if it is a content form or not.
- 🇨🇭Switzerland saschaeggi Zurich
@jeni_dc can you confirm the changes from @gwvoigt work as expected? Happy to merge then.
- 🇬🇧United Kingdom jeni_dc
Sorry for the late reply I only had a chance to update Gin on that site. Yes the changes are still working as expected, but that site is still on Group 2 for the time being so I can't really say for now with Group 3.
I tested this on latest dev and 8.x-3.0-rc2, both worked as expected.
- 🇨🇭Switzerland saschaeggi Zurich
Can this be tested & verified it works as well with Group v3?
- Status changed to RTBC
10 months ago 1:55pm 19 January 2024 - 🇬🇧United Kingdom jeni_dc
I just tested this with a fresh install of Group 3.2.2 with Group Term 4.1.0 on both core 10.1.8 and 10.2.2 and the add term form looked as it should. Screenshot attached.
-
saschaeggi →
committed dd2e11ca on 8.x-3.x authored by
jeni_dc →
Issue #3265646: Group content form irregularities due to content form...
-
saschaeggi →
committed dd2e11ca on 8.x-3.x authored by
jeni_dc →
- Status changed to Fixed
8 months ago 10:39pm 20 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.