feat: add role-based access checks#9

Merged

erseco merged 2 commits intomainfrom

zkxmrv-feature/add-role-checks-for-operations

Aug 5, 2025

Owner

erseco commented Aug 5, 2025

Summary

add permissions utility with role caching and @requires_role decorator
enforce manager role for course creation and admin role for user deletion
cover role checks with unit tests and relax repository ID assertion
detect manager role via course management page in permission checks

Testing

make format
make test-staging

https://chatgpt.com/codex/tasks/task_e_6891b3585d248322815f29cd68fa4ecb


          fix: detect manager role in permissions utility

1ff4842

erseco added the codex label

— with

ChatGPT Codex Connector


          Merge branch 'main' into zkxmrv-feature/add-role-checks-for-operations

0fcc040

Owner Author

erseco commented Aug 5, 2025

fix #5

erseco merged commit 9d22873 into main

6 checks passed

erseco deleted the zkxmrv-feature/add-role-checks-for-operations branch

August 5, 2025 08:50

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

Labels