Skip to content

Commit f442d3e

Browse files
feat!: Drop support for the legacy Advanced Settings page.
The legacy Advanced Settings 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.advanced_settings' waffle flag will no longer be respected. The system will behave as if the flag is set to false permanently.
1 parent 759a9bb commit f442d3e

3 files changed

Lines changed: 1 addition & 233 deletions

File tree

cms/djangoapps/contentstore/views/course.py

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,6 @@
7878
from openedx.core.djangoapps.content.course_overviews.models import CourseOverview
7979
from openedx.core.djangoapps.credit.tasks import update_credit_course_requirements
8080
from openedx.core.djangoapps.models.course_details import CourseDetails
81-
from openedx.core.djangoapps.site_configuration import helpers as configuration_helpers
8281
from openedx.core.djangolib.js_utils import dump_js_escaped_json
8382
from openedx.core.lib.api.view_utils import view_auth_classes
8483
from openedx.core.lib.course_tabs import CourseTabPluginManager
@@ -99,7 +98,6 @@
9998
from ..tasks import rerun_course as rerun_course_task
10099
from ..toggles import (
101100
default_enable_flexible_peer_openassessments,
102-
use_new_advanced_settings_page,
103101
use_new_grading_page,
104102
use_new_group_configurations_page,
105103
use_new_schedule_details_page,
@@ -115,7 +113,6 @@
115113
get_group_configurations_context,
116114
get_group_configurations_url,
117115
get_lms_link_for_item,
118-
get_proctored_exam_settings_url,
119116
get_schedule_details_url,
120117
get_studio_home_url,
121118
get_textbooks_url,
@@ -1522,24 +1519,7 @@ def advanced_settings_handler(request, course_key_string):
15221519
advanced_dict.get('mobile_available')['deprecated'] = True
15231520

15241521
if 'text/html' in request.META.get('HTTP_ACCEPT', '') and request.method == 'GET':
1525-
if use_new_advanced_settings_page(course_key):
1526-
return redirect(get_advanced_settings_url(course_key))
1527-
publisher_enabled = configuration_helpers.get_value_for_org(
1528-
course_block.location.org,
1529-
'ENABLE_PUBLISHER',
1530-
settings.FEATURES.get('ENABLE_PUBLISHER', False)
1531-
)
1532-
# gather any errors in the currently stored proctoring settings.
1533-
proctoring_errors = CourseMetadata.validate_proctoring_settings(course_block, advanced_dict, request.user)
1534-
1535-
return render_to_response('settings_advanced.html', {
1536-
'context_course': course_block,
1537-
'advanced_dict': advanced_dict,
1538-
'advanced_settings_url': reverse_course_url('advanced_settings_handler', course_key),
1539-
'publisher_enabled': publisher_enabled,
1540-
'mfe_proctored_exam_settings_url': get_proctored_exam_settings_url(course_block.id),
1541-
'proctoring_errors': proctoring_errors,
1542-
})
1522+
return redirect(get_advanced_settings_url(course_key))
15431523
elif 'application/json' in request.META.get('HTTP_ACCEPT', ''):
15441524
if request.method == 'GET':
15451525
return JsonResponse(CourseMetadata.fetch(course_block))

cms/static/js/factories/settings_advanced.js

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

cms/templates/settings_advanced.html

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

0 commit comments

Comments
 (0)