Skip to content

Conversation

@timyates
Copy link
Contributor

Currently in the back-end, we have 8 different messages when the current user's permissions are not enough to perform an operation;

  • "Requires admin privileges
  • "User is unauthorized to do this operation"
  • "You are not authorized to access this Demographics"
  • "You are not authorized to do that operation"
  • "You are not authorized to do this operation"
  • "You are not authorized to do this operation :("
  • "You are not authorized to perform this operation"
  • "You do not have permission to access this resource"

This PR consolodates all these to a single message (the same message as in the front-end)

  • "You are not authorized to perform this operation"

The value of this comes from a constant in PermissionsValidation incase we want to update across the app at a future date.

@timyates timyates requested review from mkimberlin and vhscom June 19, 2024 14:08
@timyates timyates self-assigned this Jun 19, 2024
@mkimberlin mkimberlin merged commit 47dd981 into develop Jun 19, 2024
@timyates timyates deleted the bugfix-single-message-for-unauthorized branch June 19, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants