Merge pull request #588 from edx/EDUCATOR-4363-suspicious-exam-email-…

…text update unsatisfactory exam email text to include IDV issues
openedx · Jul 12, 2019 · fd2b91c · fd2b91c
2 parents 1a3642f + d04d0b4
commit fd2b91c
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 11 deletions.
diff --git a/edx_proctoring/__init__.py b/edx_proctoring/__init__.py
@@ -5,6 +5,6 @@
 from __future__ import absolute_import
 
 # Be sure to update the version number in edx_proctoring/package.json
-__version__ = '2.0.5'
+__version__ = '2.0.6'
 
 default_app_config = 'edx_proctoring.apps.EdxProctoringConfig'  # pylint: disable=invalid-name
diff --git a/edx_proctoring/templates/emails/proctoring_attempt_satisfactory_email.html b/edx_proctoring/templates/emails/proctoring_attempt_satisfactory_email.html
@@ -2,7 +2,7 @@
 
 <p>
     {% blocktrans %}
-        Hi {{ username }},
+        Hello {{ username }},
     {% endblocktrans %}
 </p>
 <p>

diff --git a/edx_proctoring/templates/emails/proctoring_attempt_submitted_email.html b/edx_proctoring/templates/emails/proctoring_attempt_submitted_email.html
@@ -2,7 +2,7 @@
 
 <p>
     {% blocktrans %}
-        Hi {{ username }},
+        Hello {{ username }},
     {% endblocktrans %}
 </p>
 <p>

diff --git a/edx_proctoring/templates/emails/proctoring_attempt_unsatisfactory_email.html b/edx_proctoring/templates/emails/proctoring_attempt_unsatisfactory_email.html
@@ -2,17 +2,18 @@
 
 <p>
     {% blocktrans %}
-        Hi {{ username }},
+        Hello {{ username }},
     {% endblocktrans %}
 </p>
 <p>
     {% blocktrans %}
         Your proctored exam "{{ exam_name }}" in
         <a href="{{ course_url }}">{{ course_name }}</a> was reviewed and the
-        team found one or more violations of the proctored exam rules. Examples
-        of behaviors that may result in a rules violation include browsing
-        the internet, using a phone, or getting help from another person. As a
-        result of the violation(s), you did not successfully meet the proctored
+        team identified one or more violations of the proctored exam rules. Examples
+        of issues that may result in a rules violation include browsing
+        the internet, blurry or missing photo identification, using a phone,
+        or getting help from another person. As a result of the identified issue(s),
+        you did not successfully meet the proctored
         exam requirements.
     {% endblocktrans %}
 </p>

diff --git a/edx_proctoring/tests/test_email.py b/edx_proctoring/tests/test_email.py
@@ -67,7 +67,7 @@ def tearDown(self):
         [
             ProctoredExamStudentAttemptStatus.rejected,
             'Proctoring Results',
-            'the team found one or more violations',
+            'the team identified one or more violations',
         ]
     )
     @ddt.unpack
@@ -92,7 +92,7 @@ def test_send_email(self, status, expected_subject, expected_message_string):
 
         # Verify the body
         actual_body = self._normalize_whitespace(mail.outbox[0].body)
-        self.assertIn('Hi tester,', actual_body)
+        self.assertIn('Hello tester,', actual_body)
         self.assertIn('Your proctored exam "Test Exam"', actual_body)
         self.assertIn(credit_state['course_name'], actual_body)
         self.assertIn(expected_message_string, actual_body)

diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
   "name": "@edx/edx-proctoring",
   "//": "Be sure to update the version number in edx_proctoring/__init__.py",
   "//": "Note that the version format is slightly different than that of the Python version when using prereleases.",
-  "version": "2.0.4",
+  "version": "2.0.6",
   "main": "edx_proctoring/static/index.js",
   "repository": {
     "type": "git",