Commit 2157202a by Ahsan Ulhaq

Add log info for course enrollment

ECOM-4763
parent b65d245b
...@@ -616,13 +616,14 @@ class EnrollmentListView(APIView, ApiKeyPermissionMixIn): ...@@ -616,13 +616,14 @@ class EnrollmentListView(APIView, ApiKeyPermissionMixIn):
org = course_id.org org = course_id.org
update_email_opt_in(request.user, org, email_opt_in) update_email_opt_in(request.user, org, email_opt_in)
log.info('The user [%s] has already been enrolled in course run [%s].', username, course_id)
return Response(response) return Response(response)
except CourseModeNotFoundError as error: except CourseModeNotFoundError as error:
return Response( return Response(
status=status.HTTP_400_BAD_REQUEST, status=status.HTTP_400_BAD_REQUEST,
data={ data={
"message": ( "message": (
u"The course mode '{mode}' is not available for course '{course_id}'." u"The [{mode}] course mode is expired or otherwise unavailable for course run [{course_id}]."
).format(mode=mode, course_id=course_id), ).format(mode=mode, course_id=course_id),
"course_details": error.data "course_details": error.data
}) })
...@@ -634,8 +635,11 @@ class EnrollmentListView(APIView, ApiKeyPermissionMixIn): ...@@ -634,8 +635,11 @@ class EnrollmentListView(APIView, ApiKeyPermissionMixIn):
} }
) )
except CourseEnrollmentExistsError as error: except CourseEnrollmentExistsError as error:
log.warning('An enrollment already exists for user [%s] in course run [%s].', username, course_id)
return Response(data=error.enrollment) return Response(data=error.enrollment)
except CourseEnrollmentError: except CourseEnrollmentError:
log.exception("An error occurred while creating the new course enrollment for user "
"[%s] in course run [%s]", username, course_id)
return Response( return Response(
status=status.HTTP_400_BAD_REQUEST, status=status.HTTP_400_BAD_REQUEST,
data={ data={
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment