Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add following proctoring exam pages: user verification, software download, ready to start instruction #17

Merged
merged 10 commits into from
May 27, 2021

Conversation

viktorrusakov
Copy link

Added user verification, software download, ready to start instructions pages for proctoring exam flow.

OeX_Proctoring-149
OeX_Proctoring-154
OeX_Proctoring-208

Copy link
Collaborator

@idegtiarov idegtiarov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently, tests are failed, could you please fix them so we can go further in the review process.

@viktorrusakov viktorrusakov changed the base branch from sagirov/EDUCATOR-5758 to main May 26, 2021 08:52
@UvgenGen UvgenGen force-pushed the rusakov/EDUCATOR-5761/proctoring-pages branch from 00a853f to 63d12ef Compare May 26, 2021 10:56
@UvgenGen UvgenGen force-pushed the rusakov/EDUCATOR-5761/proctoring-pages branch from 63d12ef to 809b88d Compare May 26, 2021 10:56
Copy link
Collaborator

@idegtiarov idegtiarov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good so far.

@idegtiarov idegtiarov merged commit 72aede4 into main May 27, 2021
ihor-romaniuk added a commit that referenced this pull request Jun 1, 2021
…e download, ready to start instruction (#17)

* feat: add user verification page

* feat: add proctored exam instructions page (when exam is in 'ready_to_start' status)

* feat: get provider software download url from backend instead of building on the frontend

* fix: merge errors

* fix: fix tests

* feat: add verification url to user verification page

Co-authored-by: Ihor Romaniuk <[email protected]>
Co-authored-by: Sagirov Eugeniy <[email protected]>
viktorrusakov added a commit that referenced this pull request Jun 14, 2021
…ng / practice exams (#17)

* feat: check for verification only for proctored exams (previously were checkin for practice and onboarding also)

* feat: add Error, Submitted and Verified instruction pages

* test: add tests for Error, Submitted and Verified instructions

* feat: add allowProctoringOptOut to examState and use getSequenceMetadata API

* feat: Use sequence from the learning app instead of hitting the endpoint

Co-authored-by: Viktor Rusakov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants