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

Privilege assignment to Group fails #758

Closed
pdaterao opened this issue May 28, 2024 · 0 comments · Fixed by #759
Closed

Privilege assignment to Group fails #758

pdaterao opened this issue May 28, 2024 · 0 comments · Fixed by #759
Assignees

Comments

@pdaterao
Copy link

Describe the bug
Privilege assignment fails in several ways after being assigned to a group.

  • The same privilege cannot be assigned to another group
  • The privilege cannot be removed and assigned to another group
  • multi-selecting privileges in UI and saving throws errors

To Reproduce
Steps to reproduce the behavior:

New assignment

  1. Go to 'Privileges' and click on the group for which privileges have to be added
  2. Check the desired privilege and Save
  3. If this privilege was already assigned to some group, an error is thrown.

Remove assignment

  1. Go to 'Privileges' and click on the group for which privileges have to be modified
  2. Uncheck the desired privilege and click 'Save'
  3. A false success message is shown
  4. Click on "Reset" or reload the page. The old privilege is still assigned.

Expected behavior

  • Multiple groups should be able to have the same privilege.
  • One privilege should be able to be associated with multiple groups.
  • Privileges should be correctly removed from groups.

Screenshots

Screenshot 2024-05-23 at 9 06 56 AM

Desktop (please complete the following information):

  • OS: Windows Server 2019
  • Browser Chrome
  • Version OpenAM 10.8.4

Additional context
I

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants