Skip to content

Commit a20cf2b

Browse files
fix: fixed toggles
1 parent 1dc626f commit a20cf2b

3 files changed

Lines changed: 3 additions & 22 deletions

File tree

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,8 +186,7 @@ def get_use_new_group_configurations_page(self, obj):
186186
"""
187187
Method to get the use_new_group_configurations_page switch
188188
"""
189-
course_key = self.get_course_key()
190-
return toggles.use_new_group_configurations_page(course_key)
189+
return True
191190

192191
def get_enable_course_optimizer(self, obj):
193192
"""

cms/djangoapps/contentstore/toggles.py

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -365,24 +365,6 @@ def use_new_certificates_page(course_key):
365365
return not LEGACY_STUDIO_CERTIFICATES.is_enabled(course_key)
366366

367367

368-
# .. toggle_name: legacy_studio.configurations
369-
# .. toggle_implementation: WaffleFlag
370-
# .. toggle_default: False
371-
# .. toggle_description: Temporarily fall back to the old Studio Configurations page.
372-
# .. toggle_use_cases: temporary
373-
# .. toggle_creation_date: 2025-03-14
374-
# .. toggle_target_removal_date: 2025-09-14
375-
# .. toggle_tickets: https://github.com/openedx/edx-platform/issues/36275
376-
# .. toggle_warning: In Ulmo, this toggle will be removed. Only the new (React-based) experience will be available.
377-
LEGACY_STUDIO_CONFIGURATIONS = CourseWaffleFlag('legacy_studio.configurations', __name__)
378-
379-
380-
def use_new_group_configurations_page(course_key):
381-
"""
382-
Returns a boolean if new studio group configurations mfe is enabled
383-
"""
384-
return not LEGACY_STUDIO_CONFIGURATIONS.is_enabled(course_key)
385-
386368

387369
# .. toggle_name: contentstore.mock_video_uploads
388370
# .. toggle_implementation: WaffleFlag

cms/templates/widgets/header.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
from urllib.parse import quote_plus
99
from common.djangoapps.student.auth import has_studio_advanced_settings_access
1010
from cms.djangoapps.contentstore import toggles
11-
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
11+
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
1212
from openedx.core.djangoapps.discussions.config.waffle import ENABLE_PAGES_AND_RESOURCES_MICROFRONTEND
1313
from openedx.core.djangoapps.lang_pref.api import header_language_selector_is_enabled, released_languages
1414
%>
@@ -149,7 +149,7 @@ <h3 class="title"><span class="label"><span class="label-prefix sr">${_("Course"
149149
</li>
150150
% endif
151151
<li class="nav-item nav-course-settings-group-configurations">
152-
<a href="${reverse('group_configurations_list_handler', kwargs={'course_key_string': str(course_key)})}">${_("Group Configurations")}</a>
152+
<a href="${get_group_configurations_url(course_key)}">${_("Group Configurations")}</a>
153153
</li>
154154
% if mfe_proctored_exam_settings_url:
155155
<li class="nav-item nav-course-settings-exams">

0 commit comments

Comments
 (0)