Repository navigation
docs: redirect moved pages to their new location - #1348
Merged
Merged
Conversation
Each "This page has moved" stub now carries a meta refresh to the page it links to, so old URLs land on the new page without a click. The redirect lives in the stub rather than conf.py because every version is built with main's conf.py, and release branches still serve the original pages. Signed-off-by: Ivan Yang <yifanyang@nvidia.com>
Contributor
|
No actionable comments were generated in the recent review.
 |
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly and concisely describes the main change: adding redirects for documentation pages that moved to new locations. |
| Docstring Coverage | |
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (34 skipped: 3… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (34 skipped: 34 unsupported.)
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Contributor
|
✅ Preview deployed: https://NVIDIA.github.io/IsaacCapture/preview/pr-1348/ Built from commit 06017c9. It may take a minute for the updated preview to appear. |
farbod-nv
approved these changes
Oct 26, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Old docs URLs from the restructure land on a "This page has moved to …" stub, and the reader has to click through. For example:
main/getting_started/lerobot/index.html→main/integrations/lerobot/index.htmlmain/references/egocentric_hand_reconstruction.html→main/ego_post_processing/egohand.htmlThis adds a meta refresh to each of the 34 stubs on
main, pointing at the page the stub already links to, so the browser goes straight there:Sphinx emits this as
<meta http-equiv="refresh">in<head>. No JavaScript is involved, matching the docs' no-JS rule and the existing rootdocs/index.htmlredirect. The visible "has moved" link stays as the fallback.The redirect is in each stub rather than in
conf.py(e.g.sphinx-reredirects), because every version is built withmain'sconf.pyand release branches still serve the original pages at these paths.Not covered: a few pages deleted earlier without a stub are still served stale from gh-pages (
keep_files: true), e.g.device/index.html,references/device.html,references/retargeting.html. They need a target chosen per page, so they are left for a follow-up.Type of change
Testing
make current-docs(sphinx-build -W --keep-going) succeeds with no warnings.<head>; each target exists, matches the stub's visible link, and is not itself a redirect (no chains)./IsaacCapture/main/and opened each of the 34 old URLs in a browser: every one landed on the expected page.Checklist
SKIP=check-copyright-year pre-commit run --all-filesgit commit -s) per the DCOSummary by CodeRabbit