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

EDUCATOR-217-364 add logs in bulk email

Browse Source
This commit is contained in:
rabiaiftikhar
2017-05-11 15:31:00 +05:00
committed by rabia23
parent 6d1c23b7dc
commit ffa9b7efe3
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lms/djangoapps/instructor/views/api.py
View File

@@ -2523,6 +2523,7 @@ def send_email(request, course_id):
course_id = CourseKey.from_string(course_id) course_id = CourseKey.from_string(course_id)
if not BulkEmailFlag.feature_enabled(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.") return HttpResponseForbidden("Email is not enabled for this course.")
targets = json.loads(request.POST.get("send_to")) targets = json.loads(request.POST.get("send_to"))
@@ -2564,6 +2565,8 @@ def send_email(request, course_id):
from_addr=from_addr from_addr=from_addr
) )
except ValueError as err: 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)) return HttpResponseBadRequest(repr(err))
# Submit the task, so that the correct InstructorTask object gets created (for monitoring purposes) # Submit the task, so that the correct InstructorTask object gets created (for monitoring purposes)