From 14ad371ec74e28e6caead3c1291f5dd8a5f1087c Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Tue, 22 Sep 2026 11:39:24 -0400 Subject: [PATCH] refactor: drop dead ENABLE_MFE_CONFIG_API override in development.py The ENABLE_MFE_CONFIG_API toggle was removed from the platform by the DEPR in #39019, which merged on 2026-08-26. The new development.py settings merged afterwards in #37444 on 2026-09-18, so this override was already dead when it landed. Nothing reads the setting now. MFEConfigView in lms/djangoapps/mfe_config_api/views.py serves /api/mfe_config/v1 with no toggle check, so assigning the name here has no effect. The comment above it is also wrong on its own terms: it says the API is off by default in common.py, and common.py no longer defines the flag at all. The comment asked for this cleanup once the DEPR landed, so this is that follow-up. DEPR: https://github.com/openedx/openedx-platform/issues/38959 Co-Authored-By: Claude Opus 5 (1M context) --- lms/envs/development.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/lms/envs/development.py b/lms/envs/development.py index 8a1dd4ae00e5..62c577a29535 100644 --- a/lms/envs/development.py +++ b/lms/envs/development.py @@ -149,13 +149,6 @@ DISCUSSIONS_MICROFRONTEND_URL = "http://apps.local.openedx.io:2002/discussions" ADMIN_CONSOLE_MICROFRONTEND_URL = "http://apps.local.openedx.io:2025/admin-console" -# Temporarily enable the MFE Config API so the dev MFEs can fetch their runtime config from the -# LMS. It is off by default in common.py (ENABLE_MFE_CONFIG_API), which makes /api/mfe_config/v1 -# return a 404. The flag itself is being deprecated -- see -# https://github.com/openedx/openedx-platform/issues/38959. Remove this override once that DEPR -# lands, or fold its removal into the DEPR if this development.py work merges first. -ENABLE_MFE_CONFIG_API = True - # Shared backend URLs served to all MFEs via the MFE Config API (GET /api/mfe_config/v1). An MFE # that sets MFE_CONFIG_API_URL fetches this at startup and merges it *over* its build-time .env # defaults (which point at localhost), so we don't have to hand-edit each MFE's env to use the