-
Notifications
You must be signed in to change notification settings - Fork 11
create Admin controller #58
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
Comments
I think we need a Roles Controller, not admin ? |
It depends. Isnt admin the only one that can create roles? Not sure what purpose rolescontroller would have since the auth will take care of authentication |
Well admin should be a role by itself and admin should be authorized to do the creation and modification of roles ? |
scrap that we should have fixed roles ? |
Yes so on the AdminController we use Authorize and the "Admin" so only a authorized admin can reach the admin methods rn only add role since we will have mods aswell so a ModeratorController too. Authorizing is just done by adding that annotation in a nutshell. You might be overthinking it |
Admin should be able to create roles and assign roles to users.
admin should be able to use the Users controller.
The text was updated successfully, but these errors were encountered: