Skip to content

Commit 8b1b427

Browse files
feat!: Drop support for the legacy Course Team page.
The legacy Course Team page in Studio has been replaced with a new view in the Authoring MFE. This change removes the now unused JS/HTML/Python related to the old page. This work is part of #36108 BREAKING CHANGE: The 'legacy_studio.course_team' waffle flag will no longer be respected. The system will behave as if the flag is set to false permanently.
1 parent 759a9bb commit 8b1b427

4 files changed

Lines changed: 2 additions & 189 deletions

File tree

cms/djangoapps/contentstore/views/user.py

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33

44
from django.contrib.auth.decorators import login_required
55
from django.contrib.auth.models import User # lint-amnesty, pylint: disable=imported-auth-user
6-
from django.core.exceptions import PermissionDenied
76
from django.http import HttpResponseNotFound
87
from django.shortcuts import redirect
98
from django.utils.translation import gettext as _
@@ -13,15 +12,13 @@
1312
from opaque_keys.edx.locator import LibraryLocator
1413

1514
from cms.djangoapps.course_creators.views import user_requested_access
16-
from common.djangoapps.edxmako.shortcuts import render_to_response
1715
from common.djangoapps.student import auth
1816
from common.djangoapps.student.auth import STUDIO_EDIT_ROLES, STUDIO_VIEW_USERS, get_user_permissions
1917
from common.djangoapps.student.models import CourseEnrollment
2018
from common.djangoapps.student.roles import CourseInstructorRole, CourseStaffRole, LibraryUserRole
2119
from common.djangoapps.util.json_request import JsonResponse, expect_json
2220

23-
from ..toggles import use_new_course_team_page
24-
from ..utils import get_course_team, get_course_team_url
21+
from ..utils import get_course_team_url
2522

2623
__all__ = ['request_course_creator', 'course_team_handler']
2724

@@ -58,9 +55,7 @@ def course_team_handler(request, course_key_string=None, email=None):
5855
if 'application/json' in request.META.get('HTTP_ACCEPT', 'application/json'):
5956
return _course_team_user(request, course_key, email)
6057
elif request.method == 'GET': # assume html
61-
if use_new_course_team_page(course_key):
62-
return redirect(get_course_team_url(course_key))
63-
return _manage_users(request, course_key)
58+
return redirect(get_course_team_url(course_key))
6459
else:
6560
return HttpResponseNotFound()
6661

@@ -74,20 +69,6 @@ def user_with_role(user, role):
7469
'role': role
7570
}
7671

77-
78-
def _manage_users(request, course_key):
79-
"""
80-
This view will return all CMS users who are editors for the specified course
81-
"""
82-
# check that logged in user has permissions to this item
83-
user_perms = get_user_permissions(request.user, course_key)
84-
if not user_perms & STUDIO_VIEW_USERS:
85-
raise PermissionDenied()
86-
87-
manage_users_context = get_course_team(request.user, course_key, user_perms)
88-
return render_to_response('manage_users.html', manage_users_context)
89-
90-
9172
@expect_json
9273
def _course_team_user(request, course_key, email):
9374
"""

cms/static/cms/js/build.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
'js/factories/group_configurations',
2525
'js/certificates/factories/certificates_page_factory',
2626
'js/factories/index',
27-
'js/factories/manage_users',
2827
'js/factories/outline',
2928
'js/factories/settings',
3029
'js/factories/settings_advanced',

cms/static/js/factories/manage_users.js

Lines changed: 0 additions & 36 deletions
This file was deleted.

cms/templates/manage_users.html

Lines changed: 0 additions & 131 deletions
This file was deleted.

0 commit comments

Comments
 (0)