From 1dc626feb91ec517dd08898486aab3a4bc3cf3c5 Mon Sep 17 00:00:00 2001
From: irfanuddinahmad <iahmad@2u.com>
Date: Thu, 23 Apr 2026 12:06:27 +0500
Subject: [PATCH 1/3] feat!: Drop support for the legacy Group Configurations
 page.

The legacy Group Configurations 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 https://github.com/openedx/edx-platform/issues/36108

BREAKING CHANGE: The 'legacy_studio.configurations' waffle flag will
no longer be respected. The system will behave as if the flag is set to
false permanently.
---
 cms/djangoapps/contentstore/utils.py          |  12 +-
 cms/djangoapps/contentstore/views/course.py   |   7 +-
 .../views/tests/test_exam_settings_view.py    |   2 -
 .../views/tests/test_group_configurations.py  |  22 +--
 cms/static/cms/js/build.js                    |   1 -
 .../js/factories/group_configurations.js      |  35 -----
 cms/templates/group_configurations.html       | 140 ------------------
 7 files changed, 7 insertions(+), 212 deletions(-)
 delete mode 100644 cms/static/js/factories/group_configurations.js
 delete mode 100644 cms/templates/group_configurations.html

diff --git a/cms/djangoapps/contentstore/utils.py b/cms/djangoapps/contentstore/utils.py
index 35818125efab..6d77b163241e 100644
--- a/cms/djangoapps/contentstore/utils.py
+++ b/cms/djangoapps/contentstore/utils.py
@@ -46,7 +46,6 @@
     use_new_course_team_page,
     use_new_export_page,
     use_new_grading_page,
-    use_new_group_configurations_page,
     use_new_import_page,
     use_new_schedule_details_page,
     use_new_unit_page,
@@ -495,13 +494,10 @@ def get_group_configurations_url(course_locator) -> str:
     """
     Gets course authoring microfrontend URL for group configurations page view.
     """
-    group_configurations_url = None
-    if use_new_group_configurations_page(course_locator):
-        mfe_base_url = get_course_authoring_url(course_locator)
-        course_mfe_url = f'{mfe_base_url}/course/{course_locator}/group_configurations'
-        if mfe_base_url:
-            group_configurations_url = course_mfe_url
-    return group_configurations_url
+    mfe_base_url = get_course_authoring_url(course_locator)
+    if mfe_base_url:
+        return f'{mfe_base_url}/course/{course_locator}/group_configurations'
+    return ''
 
 
 def get_custom_pages_url(course_locator) -> str:
diff --git a/cms/djangoapps/contentstore/views/course.py b/cms/djangoapps/contentstore/views/course.py
index 3d248944bdd5..71ed24a7759c 100644
--- a/cms/djangoapps/contentstore/views/course.py
+++ b/cms/djangoapps/contentstore/views/course.py
@@ -101,7 +101,6 @@
     default_enable_flexible_peer_openassessments,
     use_new_advanced_settings_page,
     use_new_grading_page,
-    use_new_group_configurations_page,
     use_new_schedule_details_page,
 )
 from ..utils import (
@@ -112,7 +111,6 @@
     get_course_rerun_context,
     get_course_settings,
     get_grading_url,
-    get_group_configurations_context,
     get_group_configurations_url,
     get_lms_link_for_item,
     get_proctored_exam_settings_url,
@@ -1873,10 +1871,7 @@ def group_configurations_list_handler(request, course_key_string):
         course = get_course_and_check_manage_group_configurations_access(course_key, request.user)
 
         if 'text/html' in request.META.get('HTTP_ACCEPT', 'text/html'):
-            if use_new_group_configurations_page(course_key):
-                return redirect(get_group_configurations_url(course_key))
-            group_configurations_context = get_group_configurations_context(course, store)
-            return render_to_response('group_configurations.html', group_configurations_context)
+            return redirect(get_group_configurations_url(course_key))
         elif "application/json" in request.META.get('HTTP_ACCEPT'):
             if request.method == 'POST':
                 # create a new group configuration for the course
diff --git a/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py b/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
index ef1d7f945dd7..8b636b0252b7 100644
--- a/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
+++ b/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
@@ -53,7 +53,6 @@ def _get_exam_settings_alert_text(raw_html_content):
     @ddt.data(
         "certificates_list_handler",
         "settings_handler",
-        "group_configurations_list_handler",
         "grading_handler",
         "advanced_settings_handler"
     )
@@ -70,7 +69,6 @@ def test_view_without_exam_settings_enabled(self, handler):
     @ddt.data(
         "certificates_list_handler",
         "settings_handler",
-        "group_configurations_list_handler",
         "grading_handler",
         "advanced_settings_handler"
     )
diff --git a/cms/djangoapps/contentstore/views/tests/test_group_configurations.py b/cms/djangoapps/contentstore/views/tests/test_group_configurations.py
index 83c20f3c4fbb..274a43084c38 100644
--- a/cms/djangoapps/contentstore/views/tests/test_group_configurations.py
+++ b/cms/djangoapps/contentstore/views/tests/test_group_configurations.py
@@ -9,14 +9,11 @@
 
 import ddt
 from django.test import Client
-from edx_toggles.toggles.testutils import override_waffle_flag
 from openedx_authz.constants.roles import COURSE_DATA_RESEARCHER, COURSE_STAFF
 from rest_framework import status
 
-from cms.djangoapps.contentstore import toggles
 from cms.djangoapps.contentstore.api.tests.base import BaseCourseViewTest
 from cms.djangoapps.contentstore.course_group_config import (
-    CONTENT_GROUP_CONFIGURATION_NAME,
     ENROLLMENT_SCHEME,
     GroupConfiguration,
 )
@@ -274,27 +271,12 @@ def _url(self):
         """
         return reverse_course_url('group_configurations_list_handler', self.course.id)
 
-    @override_waffle_flag(toggles.LEGACY_STUDIO_CONFIGURATIONS, True)
     def test_view_index_ok(self):
         """
-        Basic check that the groups configuration page responds correctly.
+        Basic check that the groups configuration page redirects to the MFE.
         """
-
-        # This creates a random UserPartition.
-        self.course.user_partitions = [
-            UserPartition(0, 'First name', 'First description', [Group(0, 'Group A'), Group(1, 'Group B'), Group(2, 'Group C')]),  # lint-amnesty, pylint: disable=line-too-long
-        ]
-        self.save_course()
-
-        if 'split_test' not in self.course.advanced_modules:
-            self.course.advanced_modules.append('split_test')
-            self.store.update_item(self.course, self.user.id)
-
         response = self.client.get(self._url())
-        self.assertEqual(response.status_code, 200)  # noqa: PT009
-        self.assertContains(response, 'First name', count=1)
-        self.assertContains(response, 'Group C')
-        self.assertContains(response, CONTENT_GROUP_CONFIGURATION_NAME)
+        self.assertEqual(response.status_code, 302)  # noqa: PT009
 
     def test_unsupported_http_accept_header(self):
         """
diff --git a/cms/static/cms/js/build.js b/cms/static/cms/js/build.js
index 73089ed04a76..697cab297619 100644
--- a/cms/static/cms/js/build.js
+++ b/cms/static/cms/js/build.js
@@ -21,7 +21,6 @@
             'js/factories/base',
             'js/factories/course_create_rerun',
             'js/factories/export',
-            'js/factories/group_configurations',
             'js/certificates/factories/certificates_page_factory',
             'js/factories/index',
             'js/factories/manage_users',
diff --git a/cms/static/js/factories/group_configurations.js b/cms/static/js/factories/group_configurations.js
deleted file mode 100644
index 648ad78f0be5..000000000000
--- a/cms/static/js/factories/group_configurations.js
+++ /dev/null
@@ -1,35 +0,0 @@
-define([
-    'js/collections/group_configuration', 'js/models/group_configuration', 'js/views/pages/group_configurations'
-], function(GroupConfigurationCollection, GroupConfigurationModel, GroupConfigurationsPage) {
-    'use strict';
-
-    return function(experimentsEnabled,
-        experimentGroupConfigurationsJson,
-        allGroupConfigurationJson,
-        groupConfigurationUrl,
-        courseOutlineUrl) {
-        var experimentGroupConfigurations = new GroupConfigurationCollection(
-                experimentGroupConfigurationsJson, {parse: true}
-            ),
-            allGroupConfigurations = [],
-            newGroupConfig,
-            i;
-
-        for (i = 0; i < allGroupConfigurationJson.length; i++) {
-            newGroupConfig = new GroupConfigurationModel(allGroupConfigurationJson[i],
-                {parse: true, canBeEmpty: true});
-            newGroupConfig.urlRoot = groupConfigurationUrl;
-            newGroupConfig.outlineUrl = courseOutlineUrl;
-            allGroupConfigurations.push(newGroupConfig);
-        }
-
-        experimentGroupConfigurations.url = groupConfigurationUrl;
-        experimentGroupConfigurations.outlineUrl = courseOutlineUrl;
-        new GroupConfigurationsPage({
-            el: $('#content'),
-            experimentsEnabled: experimentsEnabled,
-            experimentGroupConfigurations: experimentGroupConfigurations,
-            allGroupConfigurations: allGroupConfigurations
-        }).render();
-    };
-});
diff --git a/cms/templates/group_configurations.html b/cms/templates/group_configurations.html
deleted file mode 100644
index 607cbc965321..000000000000
--- a/cms/templates/group_configurations.html
+++ /dev/null
@@ -1,140 +0,0 @@
-<%page expression_filter="h"/>
-<%inherit file="base.html" />
-<%def name="content_groups_help_token()"><% return "content_groups" %></%def>
-<%def name="enrollment_track_help_token()"><% return "enrollment_tracks" %></%def>
-<%def name="experiment_group_configurations_help_token()"><% return "group_configurations" %></%def>
-<%namespace name='static' file='static_content.html'/>
-<%!
-from cms.djangoapps.contentstore import utils
-from django.utils.translation import gettext as _
-from openedx.core.djangolib.js_utils import (
-    dump_js_escaped_json, js_escaped_string
-)
-from openedx.core.djangolib.markup import HTML, Text
-from six.moves.urllib.parse import quote
-%>
-
-<%block name="title">${_("Group Configurations")}</%block>
-<%block name="bodyclass">is-signedin course view-group-configurations</%block>
-
-<%block name="header_extras">
-% for template_name in ["group-configuration-details", "group-configuration-editor", "group-edit", "content-group-editor", "partition-group-details", "basic-modal", "modal-button", "list"]:
-  <script type="text/template" id="${template_name}-tpl">
-    <%static:include path="js/${template_name}.underscore" />
-  </script>
-% endfor
-</%block>
-
-<%block name="requirejs">
-  require(["js/factories/group_configurations"], function(GroupConfigurationsFactory) {
-      GroupConfigurationsFactory(
-          ${should_show_experiment_groups | n, dump_js_escaped_json},
-          ${experiment_group_configurations | n, dump_js_escaped_json},
-          ${all_group_configurations | n, dump_js_escaped_json},
-          "${group_configuration_url | n, js_escaped_string}",
-          "${course_outline_url | n, js_escaped_string}",
-          ${should_show_enrollment_track | n, dump_js_escaped_json}
-      );
-  });
-</%block>
-
-<%block name="content">
-  <div class="wrapper-mast wrapper">
-    <header class="mast has-actions has-subtitle">
-      <h1 class="page-header">
-        <small class="subtitle">${_("Settings")}</small>
-        <span class="sr">&gt; </span>${_("Group Configurations")}
-      </h1>
-    </header>
-  </div>
-
-  <div class="wrapper-content wrapper">
-    <section class="content">
-
-      <article class="content-primary" role="main">
-
-        % for config in all_group_configurations:
-          <div class="wrapper-groups content-groups ${config['scheme']} ${config['id']}">
-            <h3 class="title">${config['name']}</h3>
-              <div class="ui-loading">
-                  <p><span class="spin"><span class="icon fa fa-refresh" aria-hidden="true"></span></span> <span class="copy">${_("Loading")}</span></p>
-              </div>
-          </div>
-        % endfor
-
-        % if should_show_experiment_groups:
-          <div class="wrapper-groups experiment-groups">
-            <h3 class="title">${_("Experiment Group Configurations")}</h3>
-            % if experiment_group_configurations is None:
-              <div class="notice notice-incontext notice-moduledisabled">
-                  <p class="copy">
-                      ${_("This module is disabled at the moment.")}
-                  </p>
-              </div>
-            % else:
-              <div class="ui-loading">
-                  <p><span class="spin"><span class="icon fa fa-refresh" aria-hidden="true"></span></span> <span class="copy">${_("Loading")}</span></p>
-              </div>
-            % endif
-          </div>
-        % endif
-      </article>
-      <aside class="content-supplementary" role="complementary">
-      % if should_show_enrollment_track:
-        <div class="bit">
-          <div class="enrollment-track-doc">
-            <h3 class="title-3">${_("Enrollment Track Groups")}</h3>
-            <p>${_("Enrollment track groups allow you to offer different course content to learners in each enrollment track. Learners enrolled in each enrollment track in your course are automatically included in the corresponding enrollment track group.")}</p>
-            <p>${_("On unit pages in the course outline, you can restrict access to components to learners based on their enrollment track.")}</p>
-            <p>${_("You cannot edit enrollment track groups, but you can expand each group to view details of the course content that is designated for learners in the group.")}</p>
-            <p><a href="${get_online_help_info(enrollment_track_help_token())['doc_url']} rel="noopener" target="_blank" class="button external-help-button">${_("Learn More")}</a></p>
-          </div>
-        </div>
-      % endif
-        <div class="bit">
-          <div class="content-groups-doc">
-              <h3 class="title-3">${_("Content Groups")}</h3>
-              <p>${_("If you have cohorts enabled in your course, you can use content groups to create cohort-specific courseware. In other words, you can customize the content that particular cohorts see in your course.")}</p>
-              <p>${_("Each content group that you create can be associated with one or more cohorts. In addition to making course content available to all learners, you can restrict access to some content to learners in specific content groups. Only learners in the cohorts that are associated with the specified content groups see the additional content.")}</p>
-              <p>${Text(_("Click {em_start}New content group{em_end} to add a new content group. To edit the name of a content group, hover over its box and click {em_start}Edit{em_end}. You can delete a content group only if it is not in use by a unit. To delete a content group, hover over its box and click the delete icon.")).format(em_start=HTML("<strong>"), em_end=HTML("</strong>"))}</p>
-              <p><a href="${get_online_help_info(content_groups_help_token())['doc_url']}" rel="noopener" target="_blank" class="button external-help-button">${_("Learn More")}</a></p>
-          </div>
-        </div>
-      % if should_show_experiment_groups:
-        <div class="bit">
-          <div class="experiment-groups-doc">
-            <h3 class="title-3">${_("Experiment Group Configurations")}</h3>
-            <p>${_("Use experiment group configurations if you are conducting content experiments, also known as A/B testing, in your course. Experiment group configurations define how many groups of learners are in a content experiment. When you create a content experiment for a course, you select the group configuration to use.")}</p>
-            <p>${Text(_("Click {em_start}New Group Configuration{em_end} to add a new configuration. To edit a configuration, hover over its box and click {em_start}Edit{em_end}. You can delete a group configuration only if it is not in use in an experiment. To delete a configuration, hover over its box and click the delete icon.")).format(em_start=HTML("<strong>"), em_end=HTML("</strong>"))}</p>
-            <p><a href="${get_online_help_info(experiment_group_configurations_help_token())['doc_url']}" rel="noopener"  target="_blank" class="button external-help-button">${_("Learn More")}</a></p>
-          </div>
-        </div>
-      % endif
-
-      <div class="bit">
-      % if context_course:
-        <%
-          url_encoded_course_id = quote(str(context_course.id).encode('utf-8'), safe='')
-          details_url = utils.reverse_course_url('settings_handler', context_course.id)
-          grading_url = utils.reverse_course_url('grading_handler', context_course.id)
-          course_team_url = utils.reverse_course_url('course_team_handler', context_course.id)
-          advanced_settings_url = utils.reverse_course_url('advanced_settings_handler', context_course.id)
-        %>
-      <h3 class="title-3">${_("Other Course Settings")}</h3>
-        <nav class="nav-related" aria-label="${_('Other Course Settings')}">
-          <ul>
-            <li class="nav-item"><a href="${details_url}">${_("Details & Schedule")}</a></li>
-            <li class="nav-item"><a href="${grading_url}">${_("Grading")}</a></li>
-            <li class="nav-item"><a href="${course_team_url}">${_("Course Team")}</a></li>
-            <li class="nav-item"><a href="${advanced_settings_url}">${_("Advanced Settings")}</a></li>
-            % if mfe_proctored_exam_settings_url:
-              <li class="nav-item"><a href="${mfe_proctored_exam_settings_url}">${_("Proctored Exam Settings")}</a></li>
-            % endif
-          </ul>
-        </nav>
-      % endif
-      </div>
-      </aside>
-    </section>
-  </div>
-</%block>

From a20cf2ba007ae2eeccc87517309bc3e134b23c5e Mon Sep 17 00:00:00 2001
From: irfanuddinahmad <iahmad@2u.com>
Date: Thu, 30 Apr 2026 18:27:04 +0500
Subject: [PATCH 2/3] fix: fixed toggles

---
 .../v1/serializers/course_waffle_flags.py      |  3 +--
 cms/djangoapps/contentstore/toggles.py         | 18 ------------------
 cms/templates/widgets/header.html              |  4 ++--
 3 files changed, 3 insertions(+), 22 deletions(-)

diff --git a/cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py b/cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py
index 46f8cebd140d..6320a75fa8c1 100644
--- a/cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py
+++ b/cms/djangoapps/contentstore/rest_api/v1/serializers/course_waffle_flags.py
@@ -186,8 +186,7 @@ def get_use_new_group_configurations_page(self, obj):
         """
         Method to get the use_new_group_configurations_page switch
         """
-        course_key = self.get_course_key()
-        return toggles.use_new_group_configurations_page(course_key)
+        return True
 
     def get_enable_course_optimizer(self, obj):
         """
diff --git a/cms/djangoapps/contentstore/toggles.py b/cms/djangoapps/contentstore/toggles.py
index 30a030f966a9..bd6e7b13d74f 100644
--- a/cms/djangoapps/contentstore/toggles.py
+++ b/cms/djangoapps/contentstore/toggles.py
@@ -365,24 +365,6 @@ def use_new_certificates_page(course_key):
     return not LEGACY_STUDIO_CERTIFICATES.is_enabled(course_key)
 
 
-# .. toggle_name: legacy_studio.configurations
-# .. toggle_implementation: WaffleFlag
-# .. toggle_default: False
-# .. toggle_description: Temporarily fall back to the old Studio Configurations 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_CONFIGURATIONS = CourseWaffleFlag('legacy_studio.configurations', __name__)
-
-
-def use_new_group_configurations_page(course_key):
-    """
-    Returns a boolean if new studio group configurations mfe is enabled
-    """
-    return not LEGACY_STUDIO_CONFIGURATIONS.is_enabled(course_key)
-
 
 # .. toggle_name: contentstore.mock_video_uploads
 # .. toggle_implementation: WaffleFlag
diff --git a/cms/templates/widgets/header.html b/cms/templates/widgets/header.html
index 4a8c7a142a86..313fa89b06c1 100644
--- a/cms/templates/widgets/header.html
+++ b/cms/templates/widgets/header.html
@@ -8,7 +8,7 @@
   from urllib.parse import quote_plus
   from common.djangoapps.student.auth import has_studio_advanced_settings_access
   from cms.djangoapps.contentstore import toggles
-  from cms.djangoapps.contentstore.utils import get_pages_and_resources_url, get_course_outline_url, get_course_libraries_url, get_updates_url, get_files_uploads_url, get_video_uploads_url, get_schedule_details_url, get_grading_url, get_advanced_settings_url, get_import_url, get_export_url, get_studio_home_url, get_course_team_url, get_optimizer_url
+  from cms.djangoapps.contentstore.utils import get_pages_and_resources_url, get_course_outline_url, get_course_libraries_url, get_updates_url, get_files_uploads_url, get_video_uploads_url, get_schedule_details_url, get_grading_url, get_advanced_settings_url, get_import_url, get_export_url, get_studio_home_url, get_course_team_url, get_optimizer_url, get_group_configurations_url
   from openedx.core.djangoapps.discussions.config.waffle import ENABLE_PAGES_AND_RESOURCES_MICROFRONTEND
   from openedx.core.djangoapps.lang_pref.api import header_language_selector_is_enabled, released_languages
 %>
@@ -149,7 +149,7 @@ <h3 class="title"><span class="label"><span class="label-prefix sr">${_("Course"
                   </li>
                   % endif
                   <li class="nav-item nav-course-settings-group-configurations">
-                    <a href="${reverse('group_configurations_list_handler', kwargs={'course_key_string': str(course_key)})}">${_("Group Configurations")}</a>
+                    <a href="${get_group_configurations_url(course_key)}">${_("Group Configurations")}</a>
                   </li>
                   % if mfe_proctored_exam_settings_url:
                   <li class="nav-item nav-course-settings-exams">

From f48a54b9820f2da9f3b1c87de071a521f2618dd0 Mon Sep 17 00:00:00 2001
From: irfanuddinahmad <iahmad@2u.com>
Date: Thu, 30 Apr 2026 19:51:06 +0500
Subject: [PATCH 3/3] fix: fixed tests

---
 .../contentstore/views/tests/test_exam_settings_view.py          | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py b/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
index 8b636b0252b7..c974fdf17484 100644
--- a/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
+++ b/cms/djangoapps/contentstore/views/tests/test_exam_settings_view.py
@@ -26,7 +26,6 @@
 )
 @override_waffle_flag(toggles.LEGACY_STUDIO_CERTIFICATES, True)
 @override_waffle_flag(toggles.LEGACY_STUDIO_SCHEDULE_DETAILS, True)
-@override_waffle_flag(toggles.LEGACY_STUDIO_CONFIGURATIONS, True)
 @override_waffle_flag(toggles.LEGACY_STUDIO_GRADING, True)
 @override_waffle_flag(toggles.LEGACY_STUDIO_ADVANCED_SETTINGS, True)
 class TestExamSettingsView(CourseTestCase, UrlResetMixin):