Skip to content

Commit cf6f212

Browse files
refactor: consider course_id when migrating backward for course-level roles
1 parent 29d4f22 commit cf6f212

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

openedx_authz/engine/utils.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -348,9 +348,8 @@ def migrate_authz_to_legacy_course_roles(
348348
"role": COURSE_ROLE_EQUIVALENCES[role_external_key],
349349
}
350350

351-
# Here we prioritize course_id over org for scope since course-level scope is more specific
352-
# and also both are not needed to create a valid CourseAccessRole entry
353351
if isinstance(role_assignment.scope, CourseOverviewData):
352+
course_access_role_kwargs["org"] = role_assignment.scope.org
354353
course_access_role_kwargs["course_id"] = scope_external_key
355354
elif isinstance(role_assignment.scope, OrgCourseOverviewGlobData):
356355
course_access_role_kwargs["org"] = role_assignment.scope.org

0 commit comments

Comments
 (0)