[BD-26] Fix a bug with JS worker #14
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ng exam instructions (#14) Co-authored-by: Ihor Romaniuk <[email protected]> Co-authored-by: Sagirov Evgeniy <[email protected]>
…#16) * feat: Cover timed exam functionality with unit and functional tests
… and support for js workers from provider (#20) * feat: add handling of exam API errors * feat: add support for workers from proctoring providers
…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]>
* feat: add prerequisites pages
* feat: [BD-26] Add missed proctored exam instruction pages add test for ready to resume and error instructions
…all timed and proctored instructions (#21) * feat: add test coverage for all timed and proctored instructions Co-authored-by: Ihor Romaniuk <[email protected]>
…ovider when starting proctored exam
openedx-webhooks
added
blended
|
Thanks for the pull request, @viktorrusakov! I've created BLENDED-861 to keep track of it in Jira. More details are on the BD-26 project page. When this pull request is ready, tag your edX technical lead. |
zacharis278
pushed a commit
that referenced
this pull request
Jun 3, 2021
…proctored exams (#10) * feat: [OeX_Proctoring-178] add mock api for fetching proctoring settings and instructions * feat: use API instead of mocked data to get proctoring settings * feat: add proctoring software download page + refactor footer * feat: add user verification page * feat: add proctored exam instructions page (when exam is in 'ready_to_start' status) * feat: [OeX_Proctoring-178] fetch proctoring settings and add proctoring exam instructions (#14) Co-authored-by: Ihor Romaniuk <[email protected]> Co-authored-by: Sagirov Evgeniy <[email protected]> * [BD-26] Cover timed exam functionality with unit and functional tests (#16) * feat: Cover timed exam functionality with unit and functional tests * fix: don't do pulling when exam is in 'ready_to_submit' status (#19) * [BD-26][EDUCATOR-5791, EDUCATOR-5767] Add handling of exam API errors and support for js workers from provider (#20) * feat: add handling of exam API errors * feat: add support for workers from proctoring providers * 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]> Co-authored-by: Ihor Romaniuk <[email protected]> Co-authored-by: Sagirov Evgeniy <[email protected]>
|
can you rebase this one so commits from #11 are included? |
|
closing this one since it's part of #15 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes a bug where JS worker would not send onStartAttempt message to proctoring backend provider when proctoring exam starts.
NOTE: should be merged after #11 and #12 are both merged