From 7b39e7d89430f208b00be7b2c88eeb5805eacc62 Mon Sep 17 00:00:00 2001
From: hsinkoff <10408711+hsinkoff@users.noreply.github.com>
Date: Fri, 19 Jan 2024 20:47:17 +0000
Subject: [PATCH] fix: remove redundant code

---
 lms/djangoapps/teams/api.py | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/lms/djangoapps/teams/api.py b/lms/djangoapps/teams/api.py
index 29c6ee7d0faa..1a929c3bfcd4 100644
--- a/lms/djangoapps/teams/api.py
+++ b/lms/djangoapps/teams/api.py
@@ -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