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

Give more competition permissions to Regional Delegates #10081

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

Conversation

gregorbg
Copy link
Member

With this PR, Regional Delegates will have permission to...

  1. Access the 'Admin' part of the Competitions Overview page (which Senior Delegates already have access to)
  2. Manage competitions of their subordinate Delegates (most notably, the "Edit Competition" form) the same way that Senior Delegates already can

In short, this PR gives Regional Delegates some access rights which Senior Delegates already have.

Fixes #10073
Fixes #10072

@EdHollingdale
Copy link
Contributor

Why do Regional Delegates need more permissions? The website should be reducing the number of people with various permissions rather than increasing it IMO.

@mirandapedro
Copy link
Contributor

Thank you so much for working on this, Gregor!

Ed, please see the issues Gregor mentioned for more details. In short, as a regional delegate, I feel like these permissions would be useful for:

  • allowing regional delegates to help delegates in their region whenever necessary (e.g. reviewing details of a competition before it's confirmed)
  • giving oversight tools to the regional delegate (e.g. making sure deadlines are met, easily accessing information for promotions)

@EdHollingdale
Copy link
Contributor

Thanks Pedro. They seem like sensible reasons - now that you point it out I think Regional Delegates being able to help via a review of a competition before it is announced makes sense. As does having more visibility of the upcoming competitions for their subordinate Delegates.

@KerrieJarman
Copy link

The Board is supportive of this change

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