fix: Add account url in dynamic config

frontend-component-header since version v.3.5 requires the the settings of ACCOUNT_SETTINGS_URL to exists as it uses it point to the account MFE. - openedx/frontend-component-header/pull/210 PR that add the setting the componenet - openedx/wg-build-test-release/issues/268 which this fixes
ghassanmas · May 16, 2023 · 8a2ccc0 · 8a2ccc0
1 parent 9cdd6ef
commit 8a2ccc0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/tutormfe/patches/openedx-lms-development-settings b/tutormfe/patches/openedx-lms-development-settings
@@ -19,6 +19,9 @@ MFE_CONFIG = {
     "STUDIO_BASE_URL": "http://{{ CMS_HOST }}:8001",
     "USER_INFO_COOKIE_NAME": "user-info",
     "ACCESS_TOKEN_COOKIE_NAME": "edx-jwt-cookie-header-payload",
+{% if MFE_ACCOUNT_MFE_APP %}
+    "ACCOUNT_SETTINGS_URL": "{{ "https" if ENABLE_HTTPS else "http" }}://{{ MFE_HOST }}/{{ MFE_ACCOUNT_MFE_APP["name"] }}",
+{%- endif %}
 }
 
 # MFE-specific settings

diff --git a/tutormfe/patches/openedx-lms-production-settings b/tutormfe/patches/openedx-lms-production-settings
@@ -19,6 +19,9 @@ MFE_CONFIG = {
     "STUDIO_BASE_URL": "{{ "https" if ENABLE_HTTPS else "http" }}://{{ CMS_HOST }}",
     "USER_INFO_COOKIE_NAME": "user-info",
     "ACCESS_TOKEN_COOKIE_NAME": "edx-jwt-cookie-header-payload",
+{% if MFE_ACCOUNT_MFE_APP %}
+    "ACCOUNT_SETTINGS_URL": "{{ "https" if ENABLE_HTTPS else "http" }}://{{ MFE_HOST }}/{{ MFE_ACCOUNT_MFE_APP["name"] }}",
+{%- endif %}
 }