Remove "Create Zendesk tickets for suspicious attempts" from Proctoring Settings#78
Remove "Create Zendesk tickets for suspicious attempts" from Proctoring Settings#78santhosh-apphelix-2u wants to merge 4 commits intorelease-ulmofrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR removes the deprecated “Create Zendesk tickets for suspicious attempts” setting from the Studio Proctoring Settings UI and ensures the setting is no longer represented in mocks, request payloads, tests, or documentation—aligning this fork with upstream Open edX behavior.
Changes:
- Removed the Zendesk tickets toggle from the Proctoring Settings UI and its associated i18n message.
- Removed
create_zendesk_ticketsfrom Studio settings payloads and from related mocks. - Updated Jest tests and README documentation to reflect the removal.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/course-unit/__mocks__/courseSectionVertical.js |
Removes create_zendesk_tickets from mocked course settings data. |
src/course-outline/__mocks__/courseOutlineIndex.js |
Removes createZendeskTickets from mocked outline settings data. |
plugins/course-apps/proctoring/messages.js |
Removes the Zendesk-related translation entry. |
plugins/course-apps/proctoring/Settings.test.jsx |
Removes Zendesk-related test setup/assertions and updates payload expectations. |
plugins/course-apps/proctoring/Settings.jsx |
Removes the Zendesk radio UI, state field, and POST payload property. |
README.rst |
Removes Zendesk requirements/configuration references and minor whitespace cleanup. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jcapphelix
left a comment
There was a problem hiding this comment.
LGTM,
But we need to ask TNL team to take a look into it as in how they want it.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * ``edx-platform`` Django settings: | ||
|
|
|
|
||
| * ``ENABLE_TAGGING_TAXONOMY_PAGES``: must be enabled (which it is by default) in order to actually enable/show the new | ||
| * ``ENABLE_TAGGING_TAXONOMY_PAGES``: must be enabled (which it is by default) in order to actually enable/show the new | ||
| Tagging/Taxonomy functionality. |
viv-helix
left a comment
There was a problem hiding this comment.
@santhosh-apphelix-2u The changes look good on my end.
@jristau1984 can you also please review this ?
Looks fine to me. |
Summary
This PR removes the "Create Zendesk tickets for suspicious attempts" option from the Studio Proctoring Settings UI.
Zendesk-based workflows for reviewing suspicious proctoring attempts have been retired. Course teams now review proctoring flags directly through the proctoring provider dashboards (e.g., Proctortrack).
These changes align the edX fork with the upstream Open edX changes where the Zendesk-based proctoring workflow was removed.
Context
During the Proctortrack deprecation work in Open edX, the legacy Zendesk ticket creation functionality for suspicious proctoring attempts was removed upstream.
Although Proctortrack remains available in the edX internal deployment, the Zendesk ticket creation option is no longer used in edX either, and its presence in the UI causes confusion for partners expecting tickets to appear in their Zendesk instances.
As a result, we are aligning the edX fork with the upstream behavior by removing this setting.
Reference ticket:
COSMO2-27 — https://2u-internal.atlassian.net/browse/COSMO2-27
Changes
create_zendesk_ticketsfrom request payloadsValidation
plugins/course-apps/proctoring/Settings.test.jsxNotes