-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add default role to users
table and make roles enum
#316
Labels
Comments
Created a dev script update to this, but will need to complement with a web app update which is to be merged at the same time. |
I think we'd deprecate the |
Josh is right! This issue can thus be closed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, in the
users
table in the database, roles are either "admin" ornull
. This forces the middleware logic to make assumptions as to what kind of role a user has, typically defaulting to "user". When adding users, we should probably make sure their role is set to a default value of "user", rather thannull
.Additionally, we may want to ensure the roles are limited only to a set of enums. Currently, the only enums would be "admin", "user", but this could conceivably expand in the future.
The text was updated successfully, but these errors were encountered: