Skip to content

fix: use ENABLE_CATALOG_MICROFRONTEND as direct setting instead of FEATURES - #288

Merged
arbrandes merged 1 commit into
overhangio:mainfrom
brian-smith-tcril:fix/catalog-mfe-setting
Apr 15, 2026
Merged

fix: use ENABLE_CATALOG_MICROFRONTEND as direct setting instead of FEATURES#288
arbrandes merged 1 commit into
overhangio:mainfrom
brian-smith-tcril:fix/catalog-mfe-setting

Conversation

@brian-smith-tcril

@brian-smith-tcril brian-smith-tcril commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace FEATURES['ENABLE_CATALOG_MICROFRONTEND'] = True with ENABLE_CATALOG_MICROFRONTEND = True in the LMS common settings patch

Context

The FEATURES dictionary is being deprecated in openedx-platform as part of the platform settings simplification effort. The catalog MFE support in tutor-mfe (added in #259) predates this change and still uses the old FEATURES dict form.

Meanwhile, openedx-platform has already moved to looking up ENABLE_CATALOG_MICROFRONTEND as a direct Django setting:

The frontend-app-catalog README also uses the direct setting form in its Tutor installation example.

Related links

Test plan

  • Verify the catalog MFE is enabled correctly with the new setting form
  • Confirm ENABLE_CATALOG_MICROFRONTEND is accessible as a top-level Django setting in LMS

🤖 Generated with Claude Code

…FEATURES

The FEATURES dictionary is being deprecated in openedx-platform.
The catalog MFE toggle should be set as a top-level Django setting.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@brian-smith-tcril
brian-smith-tcril changed the base branch from release to main April 15, 2026 14:14
@arbrandes
arbrandes merged commit d1e798b into overhangio:main Apr 15, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Triage to Done in Tutor project management Apr 15, 2026
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