Skip to content

Commit

Permalink
fix: remove redundant code
Browse files Browse the repository at this point in the history
  • Loading branch information
hsinkoff committed Jan 19, 2024
1 parent 7c94e63 commit 7b39e7d
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions lms/djangoapps/teams/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -164,11 +164,7 @@ def has_team_api_access(user, course_key, access_username=None):
Returns:
bool: True if the user has access, False otherwise.
"""
# TODO: remove role checks once course_roles is fully impelented and data is migrated
if (
has_course_staff_privileges(user, course_key) or
user.has_perm(CourseRolesPermission.MANAGE_STUDENTS.perm_name, course_key)
):
if has_course_staff_privileges(user, course_key):
return True
if has_discussion_privileges(user, course_key):
return True
Expand Down

0 comments on commit 7b39e7d

Please sign in to comment.