Skip to content

Show a clearer error message when edits to an event or eventSeries are not permitted #5225

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gjvoosten
Copy link
Collaborator

@gjvoosten gjvoosten commented Jul 8, 2025

When a superuser edits an event or eventSeries without filling in an Admin Organization (or filling in one that is not permitted), they got a standard error message that permission was denied. We now show a clearer error message in that case.

Closes AB#1389

User changes

  • None

Superuser changes

  • A clearer message "You must be a superuser of the Admin Organization" is shown when edits to an event or eventSeries are not permitted.

Admin changes

  • None.

System admin changes

  • application.yml or anet-dictionary.yml needs change
  • db needs migration
  • documentation has changed
  • graphql schema has changed

Checklist

  • described the user behavior in PR body
  • referenced/updated all related issues
  • commits follow a repo#issue: Title title format and these 7 rules
  • commits have a clean history, otherwise PR may be squash-merged
  • added and/or updated unit tests
  • added and/or updated e2e tests
  • added and/or updated data migrations
  • updated documentation
  • resolved all build errors and warnings
  • opened debt issues for anything not resolved here

@gjvoosten gjvoosten marked this pull request as ready for review July 8, 2025 09:50
@gjvoosten gjvoosten force-pushed the AB-1389-improve-event-adminOrg-error-message branch 2 times, most recently from af8b71e to 49bfc1f Compare July 10, 2025 14:40
@gjvoosten gjvoosten force-pushed the AB-1389-improve-event-adminOrg-error-message branch from 49bfc1f to 75493f1 Compare July 17, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant