Skip to content

Commit

Permalink
fix: Remove confusing language on proctoring info panel
Browse files Browse the repository at this point in the history
MST-700. Remove language that says learners with a verified onboarding can take exams. This created confusion for learners who did not realize they must also complete IDV before taking a proctored exam.
  • Loading branch information
alangsto committed May 6, 2021
1 parent b67af3c commit 2b9c7ce
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 7 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,11 @@ Change Log
Unreleased
~~~~~~~~~~

[3.8.9] - 2021-05-07
~~~~~~~~~~~~~~~~~~~~
* Update language on proctored exam info panel if learner has
a verified onboarding attempt

[3.8.8] - 2021-04-23
~~~~~~~~~~~~~~~~~~~~
* Add detailed logging of ping failures
Expand Down
2 changes: 1 addition & 1 deletion edx_proctoring/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,6 @@
"""

# Be sure to update the version number in edx_proctoring/package.json
__version__ = '3.8.8'
__version__ = '3.8.9'

default_app_config = 'edx_proctoring.apps.EdxProctoringConfig' # pylint: disable=invalid-name
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
submitted: submittedText,
verified: {
status: gettext('Verified'),
message: gettext('You can now take proctored exams in this course.')
message: gettext('Your onboarding exam has been approved in this course.')
},
rejected: {
status: gettext('Rejected'),
Expand All @@ -41,7 +41,7 @@
},
other_course_approved: {
status: gettext('Approved in Another Course'),
message: gettext('You are eligible to take proctored exams in this course.'),
message: gettext('Your onboarding exam has been approved in another course.'),
detail: gettext(
'If your device has changed, we recommend that you complete this ' +
'course\'s onboarding exam in order to ensure that your setup ' +
Expand All @@ -51,8 +51,7 @@
expiring_soon: {
status: gettext('Expiring Soon'),
message: gettext(
'Your onboarding profile has been approved in another course, ' +
'so you are eligible to take proctored exams in this course. ' +
'Your onboarding profile has been approved in another course. ' +
'However, your onboarding status is expiring soon. Please ' +
'complete onboarding again to ensure that you will be ' +
'able to continue taking proctored exams.'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -445,7 +445,7 @@ describe('ProctoredExamInfo', function() {
expect(this.proctored_exam_info.$el.find('.onboarding-status').html())
.toContain('Approved in Another Course');
expect(this.proctored_exam_info.$el.find('.onboarding-status-message').html())
.toContain('You are eligible to take proctored exams');
.toContain('Your onboarding exam has been approved in another course');
expect(this.proctored_exam_info.$el.find('.onboarding-status-detail').html())
.toContain('we recommend that you complete');
expect(this.proctored_exam_info.$el.find('.action-onboarding').html())
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -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": "3.8.8",
"version": "3.8.9",
"main": "edx_proctoring/static/index.js",
"scripts":{
"test":"gulp test"
Expand Down

0 comments on commit 2b9c7ce

Please sign in to comment.