From c87269956ad90d7fabb4d3245772f735e6318fdf Mon Sep 17 00:00:00 2001 From: Jeff LaJoie Date: Fri, 8 May 2020 08:39:03 -0400 Subject: [PATCH] Changes type of objects that get serialized, removes a problematic log statement --- openedx/core/djangoapps/schedules/resolvers.py | 5 ----- openedx/core/djangoapps/schedules/tasks.py | 6 +++--- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/openedx/core/djangoapps/schedules/resolvers.py b/openedx/core/djangoapps/schedules/resolvers.py index ecc6a52ddb..9a67710168 100644 --- a/openedx/core/djangoapps/schedules/resolvers.py +++ b/openedx/core/djangoapps/schedules/resolvers.py @@ -434,11 +434,6 @@ class CourseNextSectionUpdate(PrefixedDebugLoggerMixin, RecipientResolver): def send(self): schedules = self.get_schedules() - LOG.info( - u'Found {} emails to send for course-key: {}'.format( - len(schedules), self.course_key - ) - ) for (user, language, context, is_self_paced) in schedules: msg_type = CourseUpdate() if is_self_paced else InstructorLedCourseUpdate() msg_type.personalize( diff --git a/openedx/core/djangoapps/schedules/tasks.py b/openedx/core/djangoapps/schedules/tasks.py index 0c636b75b5..e40d92f1a8 100644 --- a/openedx/core/djangoapps/schedules/tasks.py +++ b/openedx/core/djangoapps/schedules/tasks.py @@ -213,17 +213,17 @@ class ScheduleCourseNextSectionUpdate(ScheduleMessageBaseTask): @classmethod def enqueue(cls, site, current_date, day_offset, override_recipient_email=None): - target_date = (current_date - datetime.timedelta(days=day_offset)).date() + target_datetime = (current_date - datetime.timedelta(days=day_offset)) if not cls.is_enqueue_enabled(site): cls.log_info(u'Message queuing disabled for site %s', site.domain) return - cls.log_info(u'Target date = %s', target_date.isoformat()) + cls.log_info(u'Target date = %s', target_datetime.date().isoformat()) for course_key in CourseOverview.get_all_course_keys(): task_args = ( site.id, - serialize(target_date), + serialize(target_datetime), # Need to leave as a datetime for serialization purposes here course_key, override_recipient_email, )