feat: Drop support for the legacy Group Configurations page.#38420
Draft
irfanuddinahmad wants to merge 3 commits intomasterfrom
Draft
feat: Drop support for the legacy Group Configurations page.#38420irfanuddinahmad wants to merge 3 commits intomasterfrom
irfanuddinahmad wants to merge 3 commits intomasterfrom
Conversation
irfanuddinahmad
force-pushed
the
iahmad/DEPR-36108-group-configurations
branch
from
9212cb7 to
fd2e164
Compare
irfanuddinahmad
changed the title
irfanuddinahmad
force-pushed
the
iahmad/DEPR-36108-group-configurations
branch
3 times, most recently
from
0fa4af1 to
f14fa37
Compare
52 tasks
irfanuddinahmad
added
the
create-sandbox
irfanuddinahmad
force-pushed
the
iahmad/DEPR-36108-group-configurations
branch
2 times, most recently
from
aa5f97b to
aedc8c3
Compare
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 #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.
irfanuddinahmad
force-pushed
the
iahmad/DEPR-36108-group-configurations
branch
from
aedc8c3 to
1dc626f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request removes the legacy group configurations page from the codebase, including its backend context, frontend JavaScript factory, and HTML template. The application now always redirects to the new group configurations page, streamlining the code and reducing maintenance for deprecated features.
Removal of legacy group configurations page:
group_configurations.htmltemplate, which contained the HTML and logic for the old group configurations UI.group_configurations.jsfactory, which was responsible for initializing the old group configurations frontend logic.Backend cleanup related to legacy page:
use_new_group_configurations_pageandget_group_configurations_contextincourse.py, as the old page is no longer used. [1] [2]group_configurations_list_handlerview to always redirect to the new group configurations page, eliminating the conditional logic for the old page.Testing:
302 redirect to MFE