Skip to content

feat: backport frontend site config translations to Verawood#38625

Open
arbrandes wants to merge 2 commits intoopenedx:release/verawoodfrom
arbrandes:frontend-site-config-backports
Open

feat: backport frontend site config translations to Verawood#38625
arbrandes wants to merge 2 commits intoopenedx:release/verawoodfrom
arbrandes:frontend-site-config-backports

Conversation

@arbrandes
Copy link
Copy Markdown
Contributor

Description

Backports two frontend site config translation features from master to release/verawood.

#38577 maps the legacy STUDIO_BASE_URL key to the optional cmsBaseUrl field in SITE_CONFIG_TRANSLATION_MAP, matching frontend-base's OptionalSiteConfig interface. With this change, FrontendSiteConfigView surfaces the Studio base URL at the top level of the response rather than nesting it inside commonAppConfig.

#38610 adds a structural translator from the legacy PARAGON_THEME_URLS shape (frontend-platform's env.config.jsx) into the frontend-base SiteConfig.theme setting served by FrontendSiteConfigView. The translator narrows to urls.brandOverride only, since frontend-base loads Paragon's base CSS via its shell stylesheet and the runtime theme setting is exclusively a brand-override mechanism. PARAGON_THEME_URLS is also added to the set of site-level keys stripped from per-app MFE_CONFIG_OVERRIDES.

Both commits cherry-pick cleanly with no conflicts.

LLM usage notice

Built with assistance from Claude.

arbrandes and others added 2 commits May 7, 2026 19:13
Maps the legacy STUDIO_BASE_URL key to the optional frontend-base
cmsBaseUrl field so it surfaces at the top level of FrontendSiteConfigView
instead of falling through to commonAppConfig.

Co-Authored-By: Claude <noreply@anthropic.com>
Adds a structural translator from the legacy PARAGON_THEME_URLS shape
(frontend-platform's env.config.jsx) into the frontend-base SiteConfig
`theme` setting served by FrontendSiteConfigView.

frontend-base loads Paragon's base CSS via its shell stylesheet, so the
runtime `theme` is exclusively a brand-override mechanism.  The
translator deliberately narrows to `urls.brandOverride` only; bare `url`
and `urls.default` would point at Paragon defaults and clobber the
bundled CSS, so they are dropped.  `defaults` passes through unchanged
when at least one URL survives, and is dropped (along with the whole
`theme` key) when no URL survives, since `defaults` alone is meaningless
without a stylesheet to point at.

PARAGON_THEME_URLS is also added to the site-level keys stripped from
per-app MFE_CONFIG_OVERRIDES, since `theme` is site-level only in
frontend-base.

Co-Authored-By: Claude <noreply@anthropic.com>
@arbrandes arbrandes changed the title feat: backport frontend site config translations to verawood feat: backport frontend site config translations to Verawood May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants