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

[BD-26][EDUCATOR-5791, EDUCATOR-5767] Add handling of exam API errors and support for js workers from provider #7

Conversation

viktorrusakov
Copy link
Contributor

@viktorrusakov viktorrusakov commented May 19, 2021

This PR allows user-friendly handling of exam API errors (previously the errors would just be printed in the console) and also adds support for js workers from proctoring providers

JIRA: EDUCATOR-5791 EDUCATOR-5767

@openedx-webhooks
Copy link

openedx-webhooks commented May 19, 2021

Thanks for the pull request, @viktorrusakov! I've created BLENDED-845 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.

@idegtiarov
Copy link
Contributor

idegtiarov commented May 19, 2021

@viktorrusakov Please add [BD-26] prefix to the PR's title.

@viktorrusakov viktorrusakov changed the title [EDUCATOR-5791] feat: add handling of exam API errors [BD-26][EDUCATOR-5791] feat: add handling of exam API errors May 19, 2021
@openedx-webhooks openedx-webhooks added blended PR is managed through 2U's blended developmnt program and removed open-source-contribution PR author is not from Axim or 2U labels May 19, 2021
…156/support-for-js-workers

Add support for js workers from proctoring providers
@viktorrusakov viktorrusakov changed the title [BD-26][EDUCATOR-5791] feat: add handling of exam API errors [BD-26][EDUCATOR-5791, EDUCATOR-5767] Add handling of exam API errors and support for js workers from provider May 21, 2021
ihor-romaniuk and others added 2 commits May 25, 2021 15:40
…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
@idegtiarov
Copy link
Contributor

@zacharis278 Please review )))

Copy link
Contributor

@zacharis278 zacharis278 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 👍

@github-actions
Copy link

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blended PR is managed through 2U's blended developmnt program merged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants