Description of problem: On the My Groups page, there are Remove and Add buttons which actually add or remove entire groups, even though you would expect that they are actually adding or removing your *membership* to a group. Version-Release number of selected component (if applicable): 0.17 How reproducible: easy Steps to Reproduce: 1. Go to My Groups 2. See a group you want to remove yourself from 3. Click remove Actual results: Entire group is deleted Expected results: I am removed from the group Additional info:
(In reply to Dan Callaghan from comment #0) > Description of problem: > On the My Groups page, there are Remove and Add buttons which actually add > or remove entire groups, even though you would expect that they are actually > adding or removing your *membership* to a group. Just a note that, only admins will see the Remove button. I think may be have a separate button/link for "Remove my membership" or something of that sort.
(In reply to Amit Saha from comment #4) > (In reply to Dan Callaghan from comment #0) > > Description of problem: > > On the My Groups page, there are Remove and Add buttons which actually add > > or remove entire groups, even though you would expect that they are actually > > adding or removing your *membership* to a group. > > Just a note that, only admins will see the Remove button. I think may be > have a separate button/link for "Remove my membership" or something of that > sort. While working on the patch, I realized that this is actually a new feature if we were to add it, so I have for now just renamed "Remove" to "Remove group" to fix this bug: http://gerrit.beaker-project.org/#/c/3268/
My question is do we really need the remove button on this page since system admins can delete groups on the groups page? May be we should disable it for now to avoid future confusion. Cheers,Matt Jia
Admin->Groups is only for admins, where as "My Groups" will also have groups you created or a co-owner of, and hence can remove them. Hence, if you are not an admin, there is no way you can remove a group from Admin->Groups.
A better label for the button might be "Delete Group". We use the word "remove" elsewhere but most other removals are easily undone (systems, lab controllers) whereas deleting a group is quite destructive and not easily reversible.
(In reply to Dan Callaghan from comment #8) > A better label for the button might be "Delete Group". We use the word > "remove" elsewhere but most other removals are easily undone (systems, lab > controllers) whereas deleting a group is quite destructive and not easily > reversible. Sounds a good idea to me.
This bug fix was applied to the release-0.17 branch, but we are not doing any more maintenance releases of the 0.17.x series. So this bug is fixed in Beaker 0.18.0.