fix: discussion email notification not work.

Discussion email notification relies on edx_ace and celery routing key for ace is not set properly by default, which causes the send mail task is never consumed. Close overhangio#439.
vrook-co · May 25, 2021 · 58c77ad · 58c77ad
1 parent 1b9334e
commit 58c77ad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@ Note: Breaking changes between versions are indicated by "💥".
 
 ## Unreleased
 
+- [Bugfix] Fix discussion notification not work.
 - [Improvement] Better error logging when loading a plugin from an incompatible version.
 
 ## v11.2.11 (2021-05-18)

diff --git a/tutor/templates/apps/openedx/config/lms.env.json b/tutor/templates/apps/openedx/config/lms.env.json
@@ -47,6 +47,7 @@
   "EMAIL_HOST": "{{ SMTP_HOST }}",
   "EMAIL_PORT": {{ SMTP_PORT }},
   "EMAIL_USE_TLS": {{ "true" if SMTP_USE_TLS else "false" }},
+  "ACE_ROUTING_KEY": "edx.lms.core.default",
   "HTTPS": "{{ "on" if ENABLE_HTTPS else "off" }}",
   "LANGUAGE_CODE": "{{ LANGUAGE_CODE }}",
   "SESSION_COOKIE_DOMAIN": ".{{ LMS_HOST|common_domain(CMS_HOST) }}",