-
Notifications
You must be signed in to change notification settings - Fork 92
docs: remove proctortrack related docs #1296
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
docs: remove proctortrack related docs #1296
Conversation
…k and mock provider
docs/developing.rst
Outdated
| power of the ansible plays used to construct and administer | ||
| sandboxes, e.g. those run via the ``/edx/bin/update`` script. | ||
| `More on that here.`_ | ||
| `More on that here. <https://openedx.atlassian.net/wiki/spaces/EdxOps/pages/13960183/Sandboxes#Sandboxes-Updatingcode>`_ |
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.
https://openedx.atlassian.net/wiki/spaces/EdxOps/pages/13960183/Sandboxes#Sandboxes-Updatingcode - This link doesn't exist
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.
In the earlier version also the link doesn't exist.
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.
This is the internal Link reference so wont be seen in preview mode. Anyway i have changed it to the directive prefix So you cant see it here.
Added in Line 257 - .. _More on that here.: https://openedx.atlassian.net/wiki/spaces/EdxOps/pages/13960183/Sandboxes#Sandboxes-Updatingcode
the .. prefix in RST is a directive prefix
It's telling the RST processor that whenever it sees "More on that here" in the text, it should create a hyperlink to "https://openedx.atlassian.net/wiki/spaces/EdxOps/pages/13960183/Sandboxes#Sandboxes-Updatingcode".
| you've created, and share the client_id with folks on the other end | ||
| of the integration. | ||
|
|
||
| .. _our spec: ./backends.rst |
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.
This link is not visible in the preview page
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.
This is the deleted code so it wont be visible.
Check with the exact line number in Raw and then in preview
docs/developing.rst
Outdated
| .. _system overview: ./system-overview.rst | ||
| .. _generate a public JWK keypair: https://mkjwk.org/ | ||
| .. _More on that here.: https://openedx.atlassian.net/wiki/spaces/EdxOps/pages/13960183/Sandboxes#Sandboxes-Updatingcode | ||
| .. _System Overview: ./system-overview.rst |
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.
This link is not visible in the preview page
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.
This is the Link for internal reference so wont be available in preview mode.
the .. prefix in RST is a directive prefix
It's telling the RST processor that whenever it sees "system overview_" in the text, it should create a hyperlink to the file "./system-overview.rst".
| .. image:: images/attempt_states.png | ||
|
|
||
|
|
||
| Example Action Sequence |
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.
Line 95 has Proctrotrack reference. Is this need to be handled along with the below image?
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 have changed the text slightly
| - [ ] The exam timer is shown and functions properly. [Exam Timer](#exam-timer) | ||
| - [ ] Click end my exam on the banner | ||
| - [ ] Click submit on the confirmation page | ||
| - [ ] You should see an interstitial confirming the exam has been submitted and is waiting on review |
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.
Line 102 has Proctrotrack reference? Is this to be handled?
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.
Yes, I have removed.
942cfe8 to
5cfac76
Compare
michaelroytman
left a comment
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.
This looks good so far. Please let me know when it's out of the draft state if you have made any subsequent changes so that I can re-review.
Proctortrack Documentation Cleanup Summary
As part of the Proctortrack deprecation:
Preserved Internally:
docs/developing.rst➝edx-documentation/en_us/developers/source/proctoring/proctortrack_setup.rstindex.rstunderproctoring/index.rstto include this sectionedx-proctoringrepo per guidanceRemoved (5 items):
docs/system-overview.rst– Removed outdated Proctortrack backend, JS worker, status referencesdocs/testing/test_plan.md– Removed Stage test courses, Proctortrack-specific cases & dashboard stepsNote: The preserved internal file added to the
edx-documentationrepo is currently on hold. This is due to wildcard redirects from alldevelopers/modules pointing todocs.opendex.org. We'll finalize the destination for 2U-internal content and move it accordingly in parallel.Tracking Ticket: COSMO2-112
Coordination Thread: GitHub Issue #36329
DEPR: openedx/openedx-platform#36329