enforce role 8 (cdli tablet manager) in admin access gate
Contributes #2623
Description
- enforce role 8 (
can view cdli tablet manager) inRequestAccessComponentso non-admin users with this role can access/admin/cdli-tablet; the role existed in therolestable and was assignable from the admin UI but was never checked anywhere
Type of PR
- This PR is a feature.
Technicalities
- added a role 8 whitelist block in
RequestAccessComponent::beforeFilter()allowingHomeandCdliTabletcontrollers under theAdminprefix, mirroring the existing role 2 / role 12 pattern
Checklist:
- My pull request has a descriptive title (not a vague title like "Update
index.md"). - My pull request targets the
phoenix/developbranch of the repository. - My commit messages follow best practices.
- My code follows the established code style of the repository.
- I added tests for the changes I made (if applicable).
- I added or updated documentation (if applicable).
- I tried running the project locally and verified that there are no visible errors.