Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #17690 from edx/thallada/schedules-wrong-highlights-fix

Browse Source 
EDUCATOR-2461 Schedules: don't yield an email on loop exception
This commit is contained in:
Tyler Hallada
2018-03-19 16:55:38 -04:00
committed by GitHub
parent b2fa16a3ae 80cb98affd
commit 3259ef477f
1 changed files with 12 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
openedx/core/djangoapps/schedules/resolvers.py
View File

@@ -357,20 +357,21 @@ class CourseUpdateResolver(BinnedSchedulesBaseResolver):
user, week_num, enrollment.course_id
)
)
# continue to the next schedule, don't yield an email for this one
else:
template_context.update({
'course_name': schedule.enrollment.course.display_name,
'course_url': _get_trackable_course_home_url(enrollment.course_id),
template_context.update({
'course_name': schedule.enrollment.course.display_name,
'course_url': _get_trackable_course_home_url(enrollment.course_id),
'week_num': week_num,
'week_highlights': week_highlights,
'week_num': week_num,
'week_highlights': week_highlights,
# This is used by the bulk email optout policy
'course_ids': [str(enrollment.course_id)],
})
template_context.update(_get_upsell_information_for_schedule(user, schedule))
# This is used by the bulk email optout policy
'course_ids': [str(enrollment.course_id)],
})
template_context.update(_get_upsell_information_for_schedule(user, schedule))
yield (user, schedule.enrollment.course.closest_released_language, template_context)
yield (user, schedule.enrollment.course.closest_released_language, template_context)
def _get_trackable_course_home_url(course_id):