SORMAS-Project icon indicating copy to clipboard operation
SORMAS-Project copied to clipboard

User Role: Complete Problem Message

Open KernB opened this issue 3 years ago • 1 comments

Change Description

In order for the admin to make an informed decision, he lacks information. In addition to the required right, the rights causing the message should also be displayed. This way the admin can decide whether to add the required right again or to remove the causing rights as well.

Steps to Reproduce

  1. Create a user roll from the Admin (Default) template
  2. Remove the e.g. the "View existing cases" right
  3. Try to save
  4. Have a look at the message

image

Expected Behavior

The causing / selected rights are mentioned.
A problem has occurred: The user right(s) [...] are required based on the selected ones: [ - List of rights - ]

Screenshots

System Details

  • Device: Laptop
  • SORMAS version: 1.75.0
  • Android version/Browser: Chrom
  • Server URL: https://test1401.sormas-oegd.de/
  • User Role: Admin

Additional Information

KernB avatar Sep 12 '22 11:09 KernB

Not a bug in my eyes because it's not a malfunction of the software, but an additional convenience function for the user.

MateStrysewske avatar Sep 13 '22 07:09 MateStrysewske

Since the list of user rights requiring a certain other user right can be very large, the format suggested in the issue description is not universally practical.

Additional requirements as a result of this behaviour:

  • As long as the list of user rights requiring a missing user right contains three or less elements, write them all down as suggested in the issue description
  • If there are more than three user rights requiring a missing user right, use the following format instead: '{missing user right}' required by {requiring user rights count} enabled user rights, e. g. '{first requiring user right}'

MateStrysewske avatar Nov 04 '22 12:11 MateStrysewske

Validated ticket on Local setup + test-de.sormas.netzlink.com using latest version of sormas 1.77.0-snapshot (708b537)

AndyBakcsy-she avatar Nov 14 '22 17:11 AndyBakcsy-she

Hello @dinua @AndyBakcsy-she, Sorry, I just saw that the german translation is missing even though it was translated 18 days ago. Could you check if the translations are pulled correctly? Thanks in advance. image

KristinaBorch avatar Nov 28 '22 15:11 KristinaBorch