feat: upgrade to frontend-base 2.0, add the Catalog app - #328
Merged
arbrandes merged 3 commits intoAug 29, 2026
Merged
Conversation
8 tasks
brian-smith-tcril
approved these changes
Aug 28, 2026
| @@ -0,0 +1,3 @@ | |||
| - 💥[Feature] Ship the Catalog as a frontend app (`@openedx/frontend-app-catalog`), enabled by default. The legacy Catalog MFE is no longer served: it stays declared in `MFE_APPS`, pinned to its `legacy-mfe` branch, for operators who disable the `catalog` frontend app. (by @arbrandes) | |||
| - [Improvement] Populate `FRONTEND_SITE_CONFIG["apps"]`, so that per-app runtime configuration can be set from the `mfe-lms-*-settings` patches. Unlike `commonAppConfig`, these values override the defaults an app ships. (by @arbrandes) | |||
| - [Bugfix] Set `ENABLE_CATALOG_MICROFRONTEND` in the CMS as well, so that Studio's course "about" links point at the catalog instead of the LMS. (by @arbrandes) | |||
Contributor
There was a problem hiding this comment.
Good catch!
This also got me looking into openedx-platform itself. It looks like we're past the toggle_target_removal_date for ENABLE_CATALOG_MICROFRONTEND. I'm not sure about the timing of removing the toggle and actually removing the legacy frontend code though, so I let a comment on the DEPR issue suggesting flipping that to default True openedx/openedx-platform#36785 (comment)
That being said, if we expect the actual legacy frontend removal to happen quickly then setting this here means not needing to add something to cms/envs in openedx-platform for an extremely limited time.
4 tasks
arbrandes
commented
Aug 28, 2026
arbrandes
marked this pull request as draft
August 28, 2026 09:24
arbrandes
force-pushed
the
arbrandes/catalog-frontend-app
branch
from
August 28, 2026 17:16
98ec6fb to
fd4eec4
Compare
This was
linked to
issues
Aug 29, 2026
arbrandes
force-pushed
the
arbrandes/catalog-frontend-app
branch
from
August 29, 2026 14:53
173ece7 to
4ba047c
Compare
arbrandes
changed the title
arbrandes
changed the title
arbrandes
force-pushed
the
arbrandes/catalog-frontend-app
branch
2 times, most recently
from
August 29, 2026 18:39
149cf81 to
c44a66c
Compare
arbrandes
marked this pull request as ready for review
August 29, 2026 18:39
arbrandes
force-pushed
the
arbrandes/catalog-frontend-app
branch
2 times, most recently
from
August 29, 2026 18:48
f4e24ac to
46c0fba
Compare
Include the new Catalog app, and in doing so, move the whole site to frontend-base 2.0. Authn and Learner Dashboard join the others in being enabled by default. Closes overhangio#330. Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
edx-platform migrated this toggle off FEATURES-as-dict in 635edb98dc, so `should_redirect_to_authn_microfrontend()` now reads `settings.ENABLE_AUTHN_MICROFRONTEND`. Since production.py deliberately skips FEATURES when promoting YAML keys to Django settings, the old `FEATURES['ENABLE_AUTHN_MICROFRONTEND']` assignment had become a no-op and the LMS kept serving the legacy login page. Co-Authored-By: Claude <noreply@anthropic.com>
arbrandes
force-pushed
the
arbrandes/catalog-frontend-app
branch
from
August 29, 2026 19:03
46c0fba to
f0c6599
Compare
Collaborator
Author
|
Merging this sooner rather than later, so that developers can start using it to develop the landed frontend-base conversions. |
github-project-automation
Bot
moved this from Pending Triage
to Done
in Tutor project management
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
3 participants
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.
Description
Move the site and the npm_version defaults to frontend-base 2.0 together. And with all four apps on 2.0, Authn and Learner Dashboard join Instructor Dashboard and Notifications in being enabled by default.
Catalog joins them as the fifth core app, also enabled by default.
Closes openedx/frontend-app-catalog#146.
Dependencies
This is in draft until the following is resolved, at which point the site's
package-lock.jsonwill need to be refreshed:LLM usage notice
Built with assistance from Claude.