From ffa9b7efe38e5fe11bc76f8aaefd1339cdb56588 Mon Sep 17 00:00:00 2001 From: rabiaiftikhar Date: Thu, 11 May 2017 15:31:00 +0500 Subject: [PATCH] EDUCATOR-217-364 add logs in bulk email --- lms/djangoapps/instructor/views/api.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lms/djangoapps/instructor/views/api.py b/lms/djangoapps/instructor/views/api.py index fbe6fd97ce..12dd3f5819 100644 --- a/lms/djangoapps/instructor/views/api.py +++ b/lms/djangoapps/instructor/views/api.py @@ -2523,6 +2523,7 @@ def send_email(request, course_id): course_id = CourseKey.from_string(course_id) if not BulkEmailFlag.feature_enabled(course_id): + log.warning(u'Email is not enabled for course %s', course_id) return HttpResponseForbidden("Email is not enabled for this course.") targets = json.loads(request.POST.get("send_to")) @@ -2564,6 +2565,8 @@ def send_email(request, course_id): from_addr=from_addr ) except ValueError as err: + log.exception(u'Cannot create course email for course %s requested by user %s for targets %s', + course_id, request.user, targets) return HttpResponseBadRequest(repr(err)) # Submit the task, so that the correct InstructorTask object gets created (for monitoring purposes)