fix: repoint three dead links - #1515
Conversation
- caliper_events.rst and xapi_events.rst pointed at docs/event-mapping/Supported_events.rst in event-routing-backends. That file moved to docs/technical_documentation/concepts/event-mapping/supported_events.rst and the old path 404s. - quince.rst linked github.com/openedx/edxplatform, which is not a repository. The pinned commit and path are valid under openedx/edx-platform.
|
Thanks for the pull request, @rawsun007! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Submit a signed contributor agreement (CLA)
If you've signed an agreement in the past, you may need to re-sign. Once you've signed the CLA, please allow 1 business day for it to be processed. 🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
sarina
left a comment
There was a problem hiding this comment.
Thanks for the submission! Just one change since we changed our main repo to https://github.com/openedx/openedx-platform last year.
| New Waffle Flags: | ||
|
|
||
| * `CREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE <https://github.com/openedx/edxplatform/blob/d3d203cbac609adf23a6a8f003731fef12bd1ea1/xmodule/course_block.py#L61>`__ | ||
| * `CREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE <https://github.com/openedx/edx-platform/blob/d3d203cbac609adf23a6a8f003731fef12bd1ea1/xmodule/course_block.py#L61>`__ |
There was a problem hiding this comment.
| * `CREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE <https://github.com/openedx/edx-platform/blob/d3d203cbac609adf23a6a8f003731fef12bd1ea1/xmodule/course_block.py#L61>`__ | |
| * `CREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE <https://github.com/openedx/openedx-platform/blob/d3d203cbac609adf23a6a8f003731fef12bd1ea1/xmodule/course_block.py#L61>`__ |
|
Hi @rawsun007! Please let us know if you have any questions regarding submitting a CLA form. Thanks! |
|
The Individual Contributor Agreement has been signed and submitted. Flagging so the Everything else on this PR is green: the Read the Docs build passes, and the change is three URL replacements in documentation prose with no product-review implications. |
|
@rawsun007 - the CLA may take a few days to process. In the meantime, kindly address my review comment. Thank you |
Per review: the main repository was renamed to openedx/openedx-platform last year. The pinned commit and path resolve there; the old name only worked through GitHub's rename redirect.
|
Done in 7de1731 — the link now uses Understood on the CLA taking a few days — nothing needed from you there, I will leave it to process. |
|
The CLA is signed and submitted (Axim Individual CLA via Docusign, today). Flagging so the |
Found by requesting every URL under
source/— 2,025 unique links — and keeping only the dead ones with a target I could verify.caliper_events.rstandxapi_events.rstboth linkevent-routing-backends/blob/master/docs/event-mapping/Supported_events.rstfor the list of supported events. That file 404s; it now lives atdocs/technical_documentation/concepts/event-mapping/supported_events.rst(new directory, lowercased filename), which exists onmasterat 42 KB. These are the "supported Caliper events" and "supported xAPI events" targets, so the reference pages for both event formats currently dead-end.community/release_notes/quince.rstlinksgithub.com/openedx/edxplatformforCREATE_COURSE_WITH_DEFAULT_ENROLLMENT_START_DATE. There is no such repository — the hyphen is missing. The pinned commitd3d203cand the pathxmodule/course_block.pyare both valid underopenedx/edx-platform, so only the repo name changed.Dead, but I did not guess at a replacement
educators/references/course_development/files_page.rstlinksdocs.openedx.org/en/latest/educators/how-tos/manage_files.htmlfrom aseealso. Nomanage_filespage exists anywhere insource/, so I do not know whether it was renamed or never written. The:ref:Add Course Files`` line above it still resolves.developers/references/developer_guide/extending_platform/xblocks.rstlinksedx-platform/blob/master/xmodule/progress.pyfor the Progress class. That file is gone frommasterand I could not find where the class moved.site_ops/concepts/user_retirement.rstlinksuser-retirement-guide.readthedocs.io, which no longer resolves.openedx/configurationplaybook links 404, andfrontend-template-application/tree/frontend-baseis a branch that no longer exists.xblocks-adjacent docs points into a.venv/lib/python3.11/site-packages/path inside a pinnededx-platformcommit, which cannot exist in the repository tree.Happy to follow up on any of those if you tell me what they should point at.
The
docs.openedx.org/projects/...URLs that a naive sweep flags are intersphinx bases built from f-strings inconf.pyand a<repo_name>template inget-your-project-docs-on-rtd.rst, not broken links, so I left them alone.AI disclosure
This was done with AI assistance: Claude (Opus), via Claude Code. It ran the sweep, resolved the replacements and drafted this description; I reviewed the diff. Every claim above was verified through the GitHub contents API rather than the HTML pages, because github.com rate-limited the bulk sweep and a 429 reads exactly like a 404 — one candidate looked dead for that reason and was not.
Context (per the review checklist)
cask_renames-style evidence for the moved file isdocs/technical_documentation/concepts/event-mapping/supported_events.rstexisting onevent-routing-backends@master.