Skip to content

build: Default the dev MFE config API URL to local.openedx.io - #1387

Merged
brian-smith-tcril merged 1 commit into
masterfrom
feanil/default_dev_config_url
Aug 10, 2026
Merged

build: Default the dev MFE config API URL to local.openedx.io#1387
brian-smith-tcril merged 1 commit into
masterfrom
feanil/default_dev_config_url

Conversation

@feanil

@feanil feanil commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Defaults the dev npm script's MFE_CONFIG_API_URL to http://local.openedx.io:8000/api/mfe_config/v1 instead of http://localhost:8000/....

Fetching MFE config from local.openedx.io keeps the LMS, Studio, and MFEs under one *.local.openedx.io domain, which avoids localhost-specific CORS/cookie carve-outs and better matches a production setup. It works with Tutor dev (whose default LMS host is local.openedx.io) and with the upcoming openedx-platform bare-metal development.py config.

Motivated by openedx/openedx-platform#37444.

Fetching MFE config from local.openedx.io:8000 instead of localhost:8000
keeps the LMS, Studio, and MFEs under one *.local.openedx.io domain, which
avoids localhost-specific CORS/cookie carve-outs and better matches a
production setup.

Works with Tutor dev (whose default LMS host is local.openedx.io) and with
the upcoming openedx-platform bare-metal development.py config.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.17%. Comparing base (1bf0acc) to head (c0297d2).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1387   +/-   ##
=======================================
  Coverage   81.17%   81.17%           
=======================================
  Files          51       51           
  Lines         834      834           
  Branches      241      242    +1     
=======================================
  Hits          677      677           
  Misses        150      150           
  Partials        7        7           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@feanil
feanil marked this pull request as ready for review August 7, 2026 20:27
@brian-smith-tcril
brian-smith-tcril merged commit e05993c into master Aug 10, 2026
8 of 12 checks passed
@brian-smith-tcril
brian-smith-tcril deleted the feanil/default_dev_config_url branch August 10, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants