feat!: Drop the legacy studio updates page. (#37544)

Remove the legacy studio updates page and its related artifacts. They
have been replaced by API and a new UI in the authoring MFE.

This cleanup is a part of #36108

BREAKING CHANGE: The 'legacy_studio.updates' waffle flag will no longer
be respected. The system will behave as if the flag is set to false
permanently.

Author: feanil

cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py

@@ -80,10 +80,15 @@ def get_use_new_grading_page(self, obj):
 
     def get_use_new_updates_page(self, obj):
         """
-        Method to get the use_new_updates_page switch
+        Method to indicate if we should use the new updates_page
+
+        This used to be based on a waffle flag but the flag is being removed so we
+        default it to true for now until we can remove the need for it from the consumers
+        of this serializer and the related APIs.
+
+        See https://github.com/openedx/edx-platform/issues/37497
         """
-        course_key = self.get_course_key()
-        return toggles.use_new_updates_page(course_key)
+        return True
 
     def get_use_new_import_page(self, obj):
         """

cms/djangoapps/contentstore/tests/test_contentstore.py

@@ -1498,8 +1498,6 @@ def test_get_json(handler):
             test_get_html('export_handler')
         with override_waffle_flag(toggles.LEGACY_STUDIO_COURSE_TEAM, True):
             test_get_html('course_team_handler')
-        with override_waffle_flag(toggles.LEGACY_STUDIO_UPDATES, True):
-            test_get_html('course_info_handler')
         with override_waffle_flag(toggles.LEGACY_STUDIO_CUSTOM_PAGES, True):
             test_get_html('tabs_handler')
         with override_waffle_flag(toggles.LEGACY_STUDIO_SCHEDULE_DETAILS, True):
@@ -1510,6 +1508,16 @@ def test_get_json(handler):
             test_get_html('advanced_settings_handler')
         test_get_json('textbooks_list_handler')
 
+        # Test that studio updates load
+        course_updates_url = reverse(
+            'course_info_update_handler',
+            kwargs={
+                'course_key_string': str(course_key),
+            }
+        )
+        resp = self.client.get(course_updates_url)
+        assert resp.status_code == 200
+
         # go look at the Edit page
         unit_key = course_key.make_usage_key('vertical', 'test_vertical')
         with override_waffle_flag(toggles.LEGACY_STUDIO_UNIT_EDITOR, True):

cms/djangoapps/contentstore/tests/test_course_settings.py

@@ -167,7 +167,6 @@ def test_discussion_fields_available(self, is_pages_and_resources_enabled,
     @override_waffle_flag(toggles.LEGACY_STUDIO_IMPORT, True)
     @override_waffle_flag(toggles.LEGACY_STUDIO_EXPORT, True)
     @override_waffle_flag(toggles.LEGACY_STUDIO_COURSE_TEAM, True)
-    @override_waffle_flag(toggles.LEGACY_STUDIO_UPDATES, True)
     @override_waffle_flag(toggles.LEGACY_STUDIO_CUSTOM_PAGES, True)
     @override_waffle_flag(toggles.LEGACY_STUDIO_SCHEDULE_DETAILS, True)
     @override_waffle_flag(toggles.LEGACY_STUDIO_GRADING, True)
@@ -185,7 +184,6 @@ def test_disable_advanced_settings_feature(self, disable_advanced_settings):
                 'import_handler',
                 'export_handler',
                 'course_team_handler',
-                'course_info_handler',
                 'tabs_handler',
                 'settings_handler',
                 'grading_handler',

cms/djangoapps/contentstore/toggles.py

@@ -256,25 +256,6 @@ def use_new_grading_page(course_key):
     return not LEGACY_STUDIO_GRADING.is_enabled(course_key)
 
 
-# .. toggle_name: legacy_studio.updates
-# .. toggle_implementation: WaffleFlag
-# .. toggle_default: False
-# .. toggle_description: Temporarily fall back to the old Studio Course Updates page.
-# .. toggle_use_cases: temporary
-# .. toggle_creation_date: 2025-03-14
-# .. toggle_target_removal_date: 2025-09-14
-# .. toggle_tickets: https://github.com/openedx/edx-platform/issues/36275
-# .. toggle_warning: In Ulmo, this toggle will be removed. Only the new (React-based) experience will be available.
-LEGACY_STUDIO_UPDATES = CourseWaffleFlag('legacy_studio.updates', __name__)
-
-
-def use_new_updates_page(course_key):
-    """
-    Returns a boolean if new studio updates mfe is enabled
-    """
-    return not LEGACY_STUDIO_UPDATES.is_enabled(course_key)
-
-
 # .. toggle_name: legacy_studio.import
 # .. toggle_implementation: WaffleFlag
 # .. toggle_default: False

cms/djangoapps/contentstore/utils.py

@@ -51,7 +51,6 @@
     use_new_import_page,
     use_new_schedule_details_page,
     use_new_unit_page,
-    use_new_updates_page,
     use_new_video_uploads_page,
 )
 from cms.djangoapps.models.settings.course_grading import CourseGradingModel
@@ -362,11 +361,10 @@ def get_updates_url(course_locator) -> str:
     Gets course authoring microfrontend URL for updates page view.
     """
     updates_url = None
-    if use_new_updates_page(course_locator):
-        mfe_base_url = get_course_authoring_url(course_locator)
-        course_mfe_url = f'{mfe_base_url}/course/{course_locator}/course_info'
-        if mfe_base_url:
-            updates_url = course_mfe_url
+    mfe_base_url = get_course_authoring_url(course_locator)
+    course_mfe_url = f'{mfe_base_url}/course/{course_locator}/course_info'
+    if mfe_base_url:
+        updates_url = course_mfe_url
     return updates_url
 
 

cms/djangoapps/contentstore/views/course.py

@@ -71,7 +71,6 @@
 from openedx.core.djangolib.js_utils import dump_js_escaped_json
 from openedx.core.lib.course_tabs import CourseTabPluginManager
 from organizations.models import Organization
-from xmodule.contentstore.content import StaticContent  # lint-amnesty, pylint: disable=wrong-import-order
 from xmodule.course_block import CourseBlock, CourseFields  # lint-amnesty, pylint: disable=wrong-import-order
 from xmodule.error_block import ErrorBlock  # lint-amnesty, pylint: disable=wrong-import-order
 from xmodule.modulestore import EdxJSONEncoder  # lint-amnesty, pylint: disable=wrong-import-order
@@ -90,7 +89,6 @@
 from ..tasks import rerun_course as rerun_course_task
 from ..toggles import (
     default_enable_flexible_peer_openassessments,
-    use_new_updates_page,
     use_new_advanced_settings_page,
     use_new_grading_page,
     use_new_group_configurations_page,
@@ -1064,24 +1062,7 @@ def course_info_handler(request, course_key_string):
     except InvalidKeyError:
         raise Http404  # lint-amnesty, pylint: disable=raise-missing-from
 
-    with modulestore().bulk_operations(course_key):
-        course_block = get_course_and_check_access(course_key, request.user)
-        if not course_block:
-            raise Http404
-        if use_new_updates_page(course_key):
-            return redirect(get_updates_url(course_key))
-        if 'text/html' in request.META.get('HTTP_ACCEPT', 'text/html'):
-            return render_to_response(
-                'course_info.html',
-                {
-                    'context_course': course_block,
-                    'updates_url': reverse_course_url('course_info_update_handler', course_key),
-                    'handouts_locator': course_key.make_usage_key('course_info', 'handouts'),
-                    'base_asset_url': StaticContent.get_base_url_path_for_course_assets(course_block.id),
-                }
-            )
-        else:
-            return HttpResponseBadRequest("Only supports html requests")
+    return redirect(get_updates_url(course_key))
 
 
 @login_required

cms/djangoapps/contentstore/views/tests/test_course_updates.py

@@ -5,9 +5,7 @@
 
 import json
 from opaque_keys.edx.keys import UsageKey
-from edx_toggles.toggles.testutils import override_waffle_flag
 
-from cms.djangoapps.contentstore import toggles
 from cms.djangoapps.contentstore.tests.test_course_settings import CourseTestCase
 from cms.djangoapps.contentstore.utils import reverse_course_url, reverse_usage_url
 from openedx.core.lib.xblock_utils import get_course_update_items
@@ -23,7 +21,6 @@ def create_update_url(self, provided_id=None, course_key=None):
         return reverse_course_url('course_info_update_handler', course_key, kwargs=kwargs)
 
     # The do all and end all of unit test cases.
-    @override_waffle_flag(toggles.LEGACY_STUDIO_UPDATES, True)
     def test_course_update(self):
         """Go through each interface and ensure it works."""
         def get_response(content, date):
@@ -40,11 +37,6 @@ def get_response(content, date):
 
             return json.loads(resp.content.decode('utf-8'))
 
-        resp = self.client.get_html(
-            reverse_course_url('course_info_handler', self.course.id)
-        )
-        self.assertContains(resp, 'Course Updates', status_code=200)
-
         init_content = '<iframe width="560" height="315" src="http://www.youtube.com/embed/RocY-Jd93XU" frameborder="0">'  # lint-amnesty, pylint: disable=line-too-long
         content = init_content + '</iframe>'
         payload = get_response(content, 'January 8, 2013')

cms/static/cms/js/build.js

@@ -20,7 +20,6 @@
             'js/factories/asset_index',
             'js/factories/base',
             'js/factories/course_create_rerun',
-            'js/factories/course_info',
             'js/factories/export',
             'js/factories/group_configurations',
             'js/certificates/factories/certificates_page_factory',

cms/static/cms/js/spec/main.js

@@ -235,7 +235,6 @@
         'js/spec/models/settings_course_grader_spec',
         'js/spec/models/settings_grading_spec',
         'js/spec/models/upload_spec',
-        'js/spec/views/course_info_spec',
         'js/spec/views/metadata_edit_spec',
         'js/spec/views/upload_spec',
         'js/spec/video/transcripts/message_manager_spec',