- πΊπΈUnited States caesius
This simple patch still applies and still works as described.
- Status changed to Needs review
about 1 year ago 6:45pm 21 August 2023 - last update
about 1 year ago 9,627 pass - last update
about 1 year ago 9,627 pass - πΊπΈUnited States caesius
This still happens in Group 2.1.x at least -- I haven't tested on 3.1.x but am including a patch for it just in case.
As an aside, the "Add group content" page now has all content showing as "Group node (Content Type)". This is really redundant for groups that can only add node content and I plan to write a custom patch for my client project to ensure it only says "Content Type." I may open a separate ticket to request that the "Add group content" list only bundles alphabetically by label (referencing this issue) and separate them into separate headers, e.g.:
Group Node
> Alert
> Basic page
> Book
> EventGroup BespokeEntityType
> Adjunct
> IPA
> Stout - πΊπΈUnited States caesius
Here's a custom patch to edit "Group node (Content type)" to just "Content type" on the "Add new content" and "Add existing content" pages. It applies to both 2.1.x and 3.1.x.
This isn't intended to be committed to the module and should only be used if group nodes are the only content that can be added to groups for your project.
- πΊπΈUnited States caesius
updated patch to remove "Group node" label.
- Status changed to Needs work
about 1 year ago 1:44pm 12 September 2023 - π§πͺBelgium kristiaanvandeneynde Antwerp, Belgium
I'll note that core's node/add page sorts by content type machine name, but label probably makes more sense for the end user.
Agreed, but I also like to stay as close to core as possible behavior-wise. So While I do like the sections that @caesius suggested, I'm not sure I want to change the order to differ from core. That said, the patch here is not the right way forward as we need that label to clearly indicate we're dealing with something coming from Group.
Perhaps we can adjust the add page instead.