Skip to content

Enforce signup priority correctness - #565

Open
georgelgeback wants to merge 1 commit into
mainfrom
signup-enforce-correctness
Open

Enforce signup priority correctness#565
georgelgeback wants to merge 1 commit into
mainfrom
signup-enforce-correctness

Conversation

@georgelgeback

Copy link
Copy Markdown
Contributor

Basically this changes:

  • We no longer allow people to create or edit signups with priorities they should not be allowed to use. Previously a regular member could enter "Ordförande" as their priority even if it was not supported for the event, and if they weren't actually ordförande.
  • We now check that a user has the nollning priority they claim, previously a mentee could claim to be a mentor, or vice-versa.
  • GetMyPriorities now returns all your priorities, not just the most recent one. This probably should not matter but in the rare case someone is both a mentee and mentor, or mentor for both mission and mentor groups, this should return both their roles.

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 this pull request may close these issues.

1 participant