Skip to content
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

Non-admin users should not be able to add group(s) to circles #128

Open
dtygel opened this issue Sep 7, 2017 · 1 comment · May be fixed by #398
Open

Non-admin users should not be able to add group(s) to circles #128

dtygel opened this issue Sep 7, 2017 · 1 comment · May be fixed by #398
Assignees

Comments

@dtygel
Copy link

dtygel commented Sep 7, 2017

In Nextcloud, when an user belongs to a specific group, she cannot see other groups, even if she is admin of the group. If she is owner of the group, she can add new users, but these new users will forcibly belong to the group she is owner of. She cannot assign other groups to users she created.

With this behaviour in mind, users who are not global admins should not be able to add groups to circles. Now this is possible in circles app. Isn't it a bug?

@ArtificialOwl
Copy link
Member

I will add:

  • An option to limit the group linking to Global Admin only.
  • A filter on the search of the groups to that it only display the group the Non-Global-Admin belongs to (if the option is checked)

@ArtificialOwl ArtificialOwl self-assigned this Sep 8, 2017
tortuetorche added a commit to tortuetorche/circles that referenced this issue Apr 1, 2020
…oup members from any users

Default to `1`.

When value is set to `0` it's only possible to add groups where the
current user is a member or for global administrators.

Fix nextcloud#128

Signed-off-by: Tortue Torche <[email protected]>
tortuetorche added a commit to tortuetorche/circles that referenced this issue Apr 9, 2020
…oup members from any users

Default to `1`.

When value is set to `0` it's only possible to add groups where the
current user is a member or for global administrators.

Fix nextcloud#128

Signed-off-by: Tortue Torche <[email protected]>
tortuetorche added a commit to tortuetorche/circles that referenced this issue Apr 9, 2020
…oup members from any users

Default to `1`.

When value is set to `0` it's only possible to add groups where the
current user is a member or for global administrators.

Fix nextcloud#128

Signed-off-by: Tortue Torche <[email protected]>
@tortuetorche tortuetorche linked a pull request Apr 9, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants