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

feat: use lti for exam end #117

Merged
merged 2 commits into from
Sep 19, 2023
Merged

feat: use lti for exam end #117

merged 2 commits into from
Sep 19, 2023

Conversation

Zacharis278
Copy link
Contributor

@Zacharis278 Zacharis278 commented Sep 12, 2023

MST-2099

  • removes temporary window.postMessage for exam submission. The new proctoring app will use LTI
  • update submission flow to use a single tab

This needs to get merged in conjunction with the new Proctorio extension so I'll hold merging until that goes out.

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.03% ⚠️

Comparison is base (d09716c) 94.12% compared to head (82b170e) 94.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   94.12%   94.09%   -0.03%     
==========================================
  Files          71       71              
  Lines        1055     1050       -5     
  Branches      289      288       -1     
==========================================
- Hits          993      988       -5     
  Misses         57       57              
  Partials        5        5              
Files Changed Coverage Δ
src/data/messages/proctorio.js 100.00% <ø> (ø)
src/data/thunks.js 93.61% <ø> (-0.09%) ⬇️
...proctored_exam/SubmitProctoredExamInstructions.jsx 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Zacharis278 Zacharis278 merged commit 27dd196 into main Sep 19, 2023
6 checks passed
@Zacharis278 Zacharis278 deleted the zhancock/lti-launch branch September 19, 2023 13:24
@github-actions
Copy link

🎉 This PR is included in version 2.23.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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants