Merge pull request #486 from edx/dcs/attemptcode

Don't override the attempt_code
openedx · Dec 10, 2018 · 155dc6d · 155dc6d
2 parents 7fd2a01 + 70e5541
commit 155dc6d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/edx_proctoring/api.py b/edx_proctoring/api.py
@@ -669,8 +669,6 @@ def create_exam_attempt(exam_id, user_id, taking_as_proctored=False):
             exam,
             context=context,
         )
-        if external_id:
-            attempt_code = external_id
 
     attempt = ProctoredExamStudentAttempt.create_exam_attempt(
         exam_id,

diff --git a/edx_proctoring/backends/rest.py b/edx_proctoring/backends/rest.py
@@ -157,6 +157,8 @@ def register_exam_attempt(self, exam, context):
         url = self.create_exam_attempt_url.format(exam_id=exam['external_id'])
         payload = context
         payload['status'] = 'created'
+        # attempt code isn't needed in this API
+        payload.pop('attempt_code', False)
         log.debug('Creating exam attempt for %r at %r', exam['external_id'], url)
         response = self.session.post(url, json=payload)
         response = response.json()