feat: updated logic to specify channel name

openedx · Oct 14, 2024 · b7fc5d4 · b7fc5d4
1 parent a7becb8
commit b7fc5d4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py b/lms/djangoapps/course_goals/management/commands/goal_reminder_email.py
@@ -94,7 +94,7 @@ def send_ace_message(goal):
         options = {
             'transactional': True,
             'from_address': settings.LMS_COMM_DEFAULT_FROM_EMAIL,
-            'override_default_channel': is_ses_enabled,
+            'override_default_channel': 'django_email',
         }
 
     msg = Message(

diff --git a/lms/djangoapps/course_goals/management/commands/tests/test_goal_reminder_email.py b/lms/djangoapps/course_goals/management/commands/tests/test_goal_reminder_email.py
@@ -195,7 +195,7 @@ def test_params_with_ses(self, mock_ace):
 
             assert mock_ace.call_count == 1
             msg = mock_ace.call_args[0][0]
-            assert msg.options['override_default_channel'] is True
+            assert msg.options['override_default_channel'] == 'django_email'
             assert msg.options['from_address'] == settings.LMS_COMM_DEFAULT_FROM_EMAIL
 
     @mock.patch('lms.djangoapps.course_goals.management.commands.goal_reminder_email.ace.send')