-
Notifications
You must be signed in to change notification settings - Fork 161
feat: remove "Create Zendesk Tickets for suspicious attempts" setting from Proctored Exam Settings #2517
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
base: master
Are you sure you want to change the base?
feat: remove "Create Zendesk Tickets for suspicious attempts" setting from Proctored Exam Settings #2517
Conversation
Skipped tests related to Zendesk ticket functionality due to proctortrack provider deprecation.
Skip tests related to proctortrack provider updates.
Updated tests to skip those related to Proctortrack deprecation.
Updated test to skip due to Proctortrack deprecation and changed selector for escalation email input.
Removed tests related to Proctortrack due to its deprecation.
…rom Proctored Exam Settings
Removed 'create_zendesk_tickets' from proctored exam settings in tests.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2517 +/- ##
=======================================
Coverage 94.70% 94.71%
=======================================
Files 1202 1202
Lines 26842 26845 +3
Branches 6041 6023 -18
=======================================
+ Hits 25422 25425 +3
Misses 1350 1350
Partials 70 70 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Removed the 'create_zendesk_tickets' property from the settings object in the test.
Add logic to set form values and show escalation email based on proctoring provider selection.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left one comment. After we've discussed and addressed that, I think this will be good to go.
Removed createZendeskTickets from form values.
This PR removes the deprecated “Create Zendesk Tickets for suspicious attempts” setting from the Proctored Exam Settings modal in the
frontend-app-authoring
MFE.This option was previously used with PSI and Zendesk to generate support tickets for suspicious exam attempts. Since both systems are retired, the setting no longer serves a purpose and has been fully removed.
Technical Changes Made
1.
plugins/course-apps/proctoring/Settings.jsx
createZendeskTickets
frominitialFormValues
handleChange
to drop references tocreateZendeskTickets
software_secure
providercreate_zendesk_tickets
from the API payload (studioDataToPostBack
)Form.Group
,Form.RadioSet
) that rendered the Yes/No radio buttons2.
plugins/course-apps/proctoring/Settings.test.jsx
createZendeskTickets
UI elements and payloadsallowOptingOut
behavior3.
plugins/course-apps/proctoring/messages.js
authoring.proctoring.createzendesk.label
Ref - https://2u-internal.atlassian.net/browse/COSMO2-27