Skip to content
This repository has been archived by the owner on Jun 21, 2019. It is now read-only.

Role assignment #169

Open
wants to merge 12 commits into
base: staging
Choose a base branch
from
Open

Role assignment #169

wants to merge 12 commits into from

Conversation

shreychowdhary
Copy link
Contributor

@shreychowdhary shreychowdhary commented Nov 26, 2017

Fixes issue #108

  • Allows any NON_PROFESSIONAL or above user to modify the roles of other users
  • A user should only be able to assign or remove a role equivalent to their role or below
  • Users should not be allowed to modify the roles of users who have a role higher than their own
  • Users can't modify their own roles
  • Users cannot use this functionality using user impersonation
  • Users should not be allowed to assign staff roles to attendees, or mentors
  • Testing

@YashoSharma YashoSharma changed the base branch from master to staging December 27, 2017 02:56
@YashoSharma
Copy link
Member

I think a lot of this functionality should be an extension of the user routes we already have.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants