Merge pull request #31529 from openedx/shahbaz/VAN-1236

fix: Update enrollment email for master track learners
This commit is contained in:
Shahbaz Shabbir
2023-01-16 15:54:12 +05:00
committed by GitHub
3 changed files with 6 additions and 2 deletions

View File

@@ -596,7 +596,8 @@ class CourseEnrollment(models.Model):
self.course_overview.display_name,
self.course_overview.short_description,
self.course_overview.has_ended(),
course_pacing_type))
course_pacing_type,
self.mode))
segment_properties['email'] = self.user.email
# This next property is for an experiment, see method's comments for more information
segment_properties['external_course_updates'] = set_up_external_updates_for_enrollment(self.user,

View File

@@ -31,7 +31,7 @@ COUNTDOWN = 60
@shared_task(bind=True, ignore_result=True)
def send_course_enrollment_email(
self, user_id, course_id, course_title, short_description, course_ended, pacing_type
self, user_id, course_id, course_title, short_description, course_ended, pacing_type, track_mode
):
"""
Send course enrollment email using Braze API.
@@ -69,6 +69,7 @@ def send_course_enrollment_email(
"learning_base_url": configuration_helpers.get_value(
"LEARNING_MICROFRONTEND_URL", settings.LEARNING_MICROFRONTEND_URL
),
"track_mode": track_mode
}
try:

View File

@@ -39,6 +39,7 @@ class TestCourseEnrollmentEmailTask(ModuleStoreTestCase):
"short_description": "Short description of course",
"course_ended": False,
"pacing_type": "self-paced",
"track_mode": "audit",
}
@staticmethod
@@ -129,6 +130,7 @@ class TestCourseEnrollmentEmailTask(ModuleStoreTestCase):
"course_title": self.send_course_enrollment_email_kwargs["course_title"],
"short_description": self.send_course_enrollment_email_kwargs["short_description"],
"pacing_type": self.send_course_enrollment_email_kwargs["pacing_type"],
"track_mode": self.send_course_enrollment_email_kwargs["track_mode"],
}
if add_course_dates: