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.

Sign up for GitHub

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

Jump to bottom

refactor: rewrite the middlewares for check roles to become only one which can handle all roles #2

Open

NathanChauveau wants to merge 2 commits into CPNV-ES:main from NathanChauveau:feature/middleware-rewrited

NathanChauveau commented Jan 10, 2025

Add CheckTypeOfUser.php which will be able to handle all roles
Modify web.php
- routes which does contain GradeController as action can only be access if the user has "App\Models\Student" in the type column
- routes which does contain StudentController or StudyPlanController as action can only be access if the user has "App\Models\Manager" in the type column
Remove all traces in all 3 controllers from both deleted middlewares

NathanChauveau added 2 commits

January 10, 2025 15:26


          refactor: rewrite both student and manager middleware into one middle…

dd5d729

…ware able to handle all roles


          refactor: add StudyPlanController in the middleware under the "only m…

80c2a97

…anagers" part

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet