Skip to content

feat: add role-based access checks#8

Closed
erseco wants to merge 3 commits intomainfrom
h5qc9h-feature/add-role-checks-for-operations
Closed

feat: add role-based access checks#8
erseco wants to merge 3 commits intomainfrom
h5qc9h-feature/add-role-checks-for-operations

Conversation

@erseco
Copy link
Owner

@erseco 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

Testing

  • make test-staging

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

@erseco
Copy link
Owner Author

erseco commented Aug 5, 2025

Fix #5

@erseco erseco closed this Aug 5, 2025
@erseco erseco deleted the h5qc9h-feature/add-role-checks-for-operations branch March 17, 2026 11:38
@erseco erseco restored the h5qc9h-feature/add-role-checks-for-operations branch March 17, 2026 11:39
@erseco erseco deleted the h5qc9h-feature/add-role-checks-for-operations branch March 17, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant