diff --git a/changelog.d/20260120_142137_eemaanamir_remove_site_id.md b/changelog.d/20260120_142137_eemaanamir_remove_site_id.md deleted file mode 100644 index 263bf66b7b..0000000000 --- a/changelog.d/20260120_142137_eemaanamir_remove_site_id.md +++ /dev/null @@ -1 +0,0 @@ -- 💥[Bugfix] Stop early SITE_ID override and document correct email configuration (by @eemaanamir) diff --git a/docs/configuration.rst b/docs/configuration.rst index 82e887e068..466a2eb0a6 100644 --- a/docs/configuration.rst +++ b/docs/configuration.rst @@ -320,33 +320,6 @@ This configuration parameter sets the Contact Email. This configuration parameter sets the Platform Name. -SITE_ID and email branding -~~~~~~~~~~~~~~~~~~~~~~~~~~ - -We do **not** override ``SITE_ID`` during initial Tutor setup. - -Leaving ``SITE_ID`` unset allows Django to create sites correctly: - -- ``example.com`` → ``id = 1`` -- LMS site → ``id = 2`` - -Overriding ``SITE_ID`` too early may result in the LMS site using ``id = 3`` and -``example.com`` using ``id = 2``. - -When to set SITE_ID -******************* - -For features that rely on site context (for example bulk emails or other -personalized emails), explicitly set:: - - SITE_ID = 2 - -Add this setting in the ``openedx-common-settings`` patch:: - - patches: - openedx-common-settings: | - SITE_ID = 2 - Custom Open edX docker image ---------------------------- diff --git a/docs/troubleshooting.rst b/docs/troubleshooting.rst index 196e51e529..c99a016059 100644 --- a/docs/troubleshooting.rst +++ b/docs/troubleshooting.rst @@ -226,20 +226,3 @@ The handy :ref:`update-mysql-authentication-plugin