Skip to content

Commit 16e67e8

Browse files
committed
squash!: Added comment on legacy role mapping for clarity
1 parent 630903a commit 16e67e8

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

openedx_authz/constants/roles.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,8 @@
181181
COURSE_BETA_TESTER = RoleData(external_key="course_beta_tester", permissions=COURSE_BETA_TESTER_PERMISSIONS)
182182

183183
# Map of legacy course role names to their equivalent new roles
184+
# This mapping must be unique in both directions, since it may be used as a reverse lookup (value → key).
185+
# If multiple keys share the same value, it will lead to collisions.
184186
LEGACY_COURSE_ROLE_EQUIVALENCES = {
185187
"instructor": COURSE_ADMIN.external_key,
186188
"staff": COURSE_STAFF.external_key,

0 commit comments

Comments
 (0)