Closed (outdated)
Project:
Group
Version:
8.x-1.x-dev
Component:
Group (group)
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Reporter:
Created:
23 Mar 2016 at 20:06 UTC
Updated:
7 Jul 2020 at 09:16 UTC
Jump to comment: Most recent
Group D8 is already in a very usable state, but still needs a few items to deserve a full release.
Things to keep an eye on from the old issue summary
Comments
Comment #2
kristiaanvandeneyndeI will create tickets for each item in the above list soon enough.
Comment #3
kristiaanvandeneyndeComment #4
kristiaanvandeneyndeComment #5
kristiaanvandeneyndeComment #6
kristiaanvandeneyndeComment #7
kristiaanvandeneyndeComment #8
kristiaanvandeneyndeComment #9
kristiaanvandeneyndeComment #10
kristiaanvandeneyndeComment #11
kristiaanvandeneyndeMade amazing progress during the sprint today, thanks to all involved!
Comment #12
kristiaanvandeneyndeAdded a rather major issue regarding the database tables.
Comment #13
kristiaanvandeneyndeComment #14
kristiaanvandeneyndeComment #15
kristiaanvandeneyndeComment #16
kristiaanvandeneyndeNope nope nope nope.
Just tried this out locally and it basically allowed anyone to craft URL's to allow people to join groups without them knowing about it beforehand. Sounds like a potential can of worms so not doing it (for now?).
Comment #17
kristiaanvandeneyndeComment #18
kristiaanvandeneyndeAdded a few items that now have issue links, removed the entry regarding skipping forms.
Comment #19
kristiaanvandeneyndeComment #20
kristiaanvandeneyndeAll beta blockers are now completed. Going through the code base one last time before tagging beta1.
@followers: If you need anything API-wise or discover any critical bugs, now would be the time to say so :)
Comment #21
kristiaanvandeneyndeComment #22
kristiaanvandeneyndeAdding an issue that is really important for the UX of adding entities to a group.
Comment #23
kristiaanvandeneyndeComment #24
kristiaanvandeneyndeComment #25
kristiaanvandeneyndeUpdated the issue summary now that we are ready to cut a release candidate.
Comment #26
kristiaanvandeneyndeComment #27
worthwhileindustries commentedI'm trying to figure out how to expose permissions from other modules. It doesn't look like you have any hooks that I can use to create a submodule.
- I want to add the ability for Admins to manage their own menus or add other permissions from other modules. Do you have a tutorial on how to extend this module or point me in the right direction to do this?
- Are these permissions within the module separate from the Drupal core permissions or are you hooking into those somehow?
Comment #28
kristiaanvandeneyndeFor questions like these you should really open up a separate issue. It makes it easier for others to find the right answer when searching.
P.S.: It's defined in mymodule.group.permissions.yml
Comment #29
zenimagine commentedHave you planned email invitation as in drupal version 8. And the possibility for a group to accept or reject an application to join?
Comment #30
kristiaanvandeneyndeNot for version 8.1.0, perhaps in a 8.1.1 release.
Comment #31
perignon commentedI think you should have Documentation as a blocker for release. At a minimum, you should have a README to tell a system administrator how to setup the module. The YouTube video links in the additional resources are for a version of the module over a year ago, the module has changed and those are not 100% applicable.
Comment #32
dravenkComment #33
kristiaanvandeneyndeFixed / Closed issues are striped through automatically :)
Comment #34
kristiaanvandeneyndeComment #35
mxtIn my opinion this is a blocker also:
#2842630: Empty page when trying to create group node
Comment #36
plachAny idea on whether #2752603: Request Membership feature should be part of this roadmap?
In #6 kristiaanvandeneynde said:
Comment #37
kristiaanvandeneyndeHmm good point. I'm actually thinking of going down a slightly different route.
Phase 1
Phase 2
Group 8.2.x initial work
#2884662: Give group permission based on global role regardless of group membership
Comment #38
geek-merlinAlso note that in #2718195-40: Add a computed field for entity's group(s) a group selector widget has gotten low hanging fruit. ;-)
Comment #39
kristiaanvandeneyndeUpdated the IS to reflect #37. Might look into #38, but no promises :)
Comment #40
scotwith1tI'd like to advocate for getting #2829966: Support for Revisions on groups in the mix here. Seems like revisions should have been considered out of the box as a core competency. Revisions should be baked into everything that relates to or behaves like content, especially as core introduces stable Workflows and Layout Builder modules. Being able to track changes made to a group and its content seems like critical (8.1) functionality to me, but happily settle for seeing it at least in the roadmap for 8.2?
Comment #41
geek-merlin+1 for Revisions.
Comment #42
skowyra commented+1 for Revisions.
Comment #43
nigelcunningham commentedAre you talking about config entities needing revisioning support? If so, I have created a module "Config Entity Revisions" that implements an API module for revisioning content entities. I started with support for Webforms, and am currently doing a big rewrite / cleanup as part of implementing support for Views. I'll happily support Groups too if that will help you.
Comment #44
jidrone commentedHi NigelCunningham,
The group entity type is a Content entity type, so your module is not applicable in this case.
Now #2829966: Support for Revisions on groups is ready for review and it was splitted in three child issues to make it easier to review.
Comment #45
nigelcunningham commentedGreat; thanks!
Comment #46
kristiaanvandeneyndeAgreed, we can at the very least try to get revisions in the first full release. If it is blocked somehow, we can do it in a minor update (8.1.1) along with core 8.7.0 when it is released May 1st and comes with support for enabling revisions post-install.
Furthermore, we need to implement https://www.drupal.org/node/2973262 and fix as many @todos as possible before a full release.
Comment #47
kristiaanvandeneyndeNew release out. Unless rc3 comes with critical bugs, I expect the next release to be 8.1.0
Comment #48
dwwThanks, this is exciting progress!
Curious if/where you see #2813405: Add a field to view and edit content's groups fitting into this roadmap. @axel.rutz mentioned #2718195: Add a computed field for entity's group(s) in #38 but that's now duplicate and merged into the work at #2813405. This seems like a really important and useful feature. Any chance we could try to get that in before 8.x-1.0 final?
Comment #49
kristiaanvandeneyndeWell, any brandnew feature at this point warrants at least one more RC, delaying a full release. So I'd rather do the following:
Comment #50
xurizaemonHave attached (IMO critical) bug #3061321: Uncaught exception in Nodes and Members tabs in rc3, which breaks Group displays on sites with existing content.
Comment #51
dwwRe: #49: Okay, fair enough. Let me know if this is out of scope and we should discuss this elsewhere...
The latest patch for #2813405: Add a field to view and edit content's groups is about 98% a whole new sub module. So I don't see any BC-break or API change issues. The only real "API changes" as such are in:
src/Form/GroupSettingsForm.php(injects themoduleHandlerservice and implements a constructor andcreate()method).src/Plugin/GroupContentEnablerManagerInterface.php(adds a new method:public function getPluginIdByEntityType()).src/Plugin/GroupContentEnablerManager.php(implements the newgetPluginIdByEntityType()method).It would be really nice to have this working in the 8.x-1.x series. Looks like you're going to need another RC, anyway. Any chance you'd be willing to get this into the RC so we don't have to worry about the API changes to the main module listed above?
Thoughts?
Thanks!
-Derek
Comment #52
heddnI think #2752603: Request Membership feature is the only outstanding item remaining on this issue. Does that sound right to anyone else or are there lurking issues not tagged to this meta that also need to land?
Comment #53
dalra commentedI think #2771065: Group Node: Only authors should be able to create Gnodes for their nodes needs to be done to prevent node authors from loosing access to their nodes.
Comment #54
dww@heddn I still think #2813405: Add a field to view and edit content's groups could be somewhere on the plan. ;)
Also, the summary is making interesting use of semver-esque versions, but this project is still using BespokeVer (8.x-Y.Z) version numbers. Please note that Y in the above maps to SemVer Major and Z maps to Minor. So "Differentiate between 8.1.1 and 8.2.x" is a kinda wonky way to express things.
We should consider if it makes sense to actually move to semver for this project, now that that's supported. If so, the official release would actually be "2.0.0", and then we'd be differentiating between "2.0.x patch-level bug fixes vs. 2.1.x minor-level new features vs. 3.0.x API-breaking changes".
If not, we'd probably benefit from being more specific and accurate in the summary. E.g. "Differentiate between 8.x-1.1 and 8.x-2.x".
Thanks,
-Derek
Comment #55
dbielke1986 commentedSince #2813405: Add a field to view and edit content's groups has been moved to a separate submodule, couldn't you build an official 8.1 release now? Is there anything missing here?
Comment #56
kristiaanvandeneyndeThere are some follow-ups to #3013678: Implement the query access handling that was recently added to Entity API that require a new Entity API release. I'm working on that now.
FYI: #3134072: Implement the query access handling for grouped entities is the blocked issue
Comment #57
kristiaanvandeneyndeClosing in preparation of a release. Will dig this up when I create a 8.x-1.1 and a 2.0.0 roadmap.