-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Temp setGroupForSession to edit current group #469
Conversation
@jburel - This seems to be working to allow the editing of your current group. If you're only a member of 1 group then editing it will fail, since you don't have any other groups you can switch to. Otherwise, editing should "just work". |
Based on my understanding of the semantics of the Web admin UIm the ability to edit a group is restricted to admins in OMERO.web. If so, shouldn't this user always be member of at least the |
@sbesson Yes, you're correct. Group owners have a Edit Group form, but that only handles permissions (not group renaming etc). Updated in 04e9538 |
Test 1 on merge-ci:
Test 2 latest-ci (trying to repeat the error which this PR is attempting to fix):
|
Fixes #163.
This uses
setGroupForSession()
to temporarily switch to a different group if a user tries to edit their current group.To test:
omero login
as an Admin which will show the name of your current group