Skip to content

chore: remove unnecessary settings and waffle flags for account and profile MFEs#246

Merged
DawoudSheraz merged 3 commits intooverhangio:mainfrom
edly-io:danyal/remove-site-configuration-script
Apr 22, 2025
Merged

chore: remove unnecessary settings and waffle flags for account and profile MFEs#246
DawoudSheraz merged 3 commits intooverhangio:mainfrom
edly-io:danyal/remove-site-configuration-script

Conversation

@Danyal-Faheem
Copy link
Contributor

Fixes part of overhangio/tutor#1207.

As the site-configuration script is planned for deprecation in tutor-core, we look to remove or update its usage with a suitable replacement.

However, as of openedx/openedx-platform#36219, we do not require the site-configuration entries for the profile MFE anymore as the MFE is the default frontend going forward.

Similarly, the redirect_to_microfrontend flags are also not required anymore for the account and profile MFE as they are now the default frontends going forward.

Copy link
Collaborator

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense!

@DawoudSheraz DawoudSheraz merged commit eccc8c2 into overhangio:main Apr 22, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Tutor project management Apr 22, 2025
@regisb regisb deleted the danyal/remove-site-configuration-script branch April 22, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants