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

[Panel] Only show changeRole option when there are multiple roles, not based on number of users #1948

Closed
GiantCrocodile opened this issue Jul 26, 2019 · 2 comments

Comments

@GiantCrocodile
Copy link

commented Jul 26, 2019

Describe the bug
If you have only 1 user, you are not able to press Change role on the users list and in the view of an individual user.

If you have > 1 users, you can change the users role, even if there is just one (admin) role.

To Reproduce
Steps to reproduce the behavior:

  1. Go to users list
  2. Click on any user
  3. Check the Change Role option
  4. Create a new user (2nd user)
  5. Repeat steps 1 to 3 and compare the behavior of change role on the users list and in an individual user view.

Expected behavior

  1. No possibility to change role if there is only one role.
  2. Same behavior regardless how many users are in the system.

Kirby Version
3.2.2

Desktop (please complete the following information):
Windows 10, latest Firefox

Additional context

  • multi-language setup with just 1 language
  • WAMP setup
@texnixe texnixe changed the title [Panel] Inconsistent change user role behavior [Panel] Only show changeRole option when there are multiple roles, not based on number of users Jul 26, 2019
@texnixe

This comment has been minimized.

Copy link
Contributor

commented Jul 26, 2019

Also: #1920

@distantnative distantnative self-assigned this Sep 14, 2019
@distantnative distantnative added this to the 3.2.5 milestone Sep 14, 2019
distantnative added a commit that referenced this issue Sep 14, 2019
distantnative added a commit that referenced this issue Sep 14, 2019
distantnative added a commit that referenced this issue Sep 15, 2019
bastianallgeier added a commit that referenced this issue Sep 16, 2019
@bastianallgeier

This comment has been minimized.

Copy link
Contributor

commented Sep 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants
You can’t perform that action at this time.