Force-redirect 33 /operate/* app-linked URLs to new-IA pages#4980
Open
shannonbradshaw wants to merge 1 commit into
Open
Force-redirect 33 /operate/* app-linked URLs to new-IA pages#4980shannonbradshaw wants to merge 1 commit into
shannonbradshaw wants to merge 1 commit into
Conversation
✅ Deploy Preview for viam-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
viambot
added
the
safe to build
The old operate/ pages still exist, so Hugo aliases on new-IA pages are shadowed by the old canonical pages. Netlify forced redirects (status 301!) override the static files, sending users to the correct new-IA pages immediately without waiting for the operate/ section to be deleted. Every redirect maps an app-linked URL from Michael's audit to the destination verified in our session. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
shannonbradshaw
force-pushed
the
force-operate-redirects
branch
from
c439d8d to
b3e7b4a
Compare
|
Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance? Auto-comment from overwatch. Will not re-nudge for 7 days. |
1 similar comment
|
Hey @shannonbradshaw — CI is green and no reviewer is assigned yet. Could you request one when you have a chance? Auto-comment from overwatch. Will not re-nudge for 7 days. |
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
2 participants
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.
Summary
The old
/operate/pages still exist on the site. Aliases on new-IA pages are shadowed because the old pages serve their canonical URLs first. This PR adds Netlify forced redirects (status = 301!) that override the old pages, sending users immediately to the correct new-IA destinations.33 redirects cover every
/operate/*URL the Viam app links to (from Michael's audit). Once this merges, the test script atcode-map/scripts/test-app-urls.pyshould show 76/76 GOOD.How forced redirects work
Normal Netlify redirects (
status = 301) only fire when no static file exists at the source path. Forced redirects (status = 301!) fire regardless, overriding any existing page.Test plan
make build-prodcleanpython3 code-map/scripts/test-app-urls.py --base-url <preview-url>against the deploy preview🤖 Generated with Claude Code