docs(redirect): deep-link the retire redirects — carry the URL fragment (PRDCT-583) - #415
Open
Iamfle4ka wants to merge 1 commit into
Open
docs(redirect): deep-link the retire redirects — carry the URL fragment (PRDCT-583)#415Iamfle4ka wants to merge 1 commit into
Iamfle4ka wants to merge 1 commit into
Conversation
A reader following a deep link like /extend/generic-extractor/configuration/api/pagination/#offset landed at the top of the help page — location.replace() used the bare target, and the meta-refresh fallback cannot forward fragments at all (browsers drop them). Append location.hash to the target unless the stub's redirect_to already pins its own anchor. Applies to every existing stub (#405/#406) and all upcoming retirements (#412/#413/#414) — the layout is shared. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Readers following old dev links with an anchor (
…/configuration/api/pagination/#offset) currently land at the top of the help page: the redirect layout callslocation.replace()with the bare target, and the meta-refresh fallback can't forward fragments (browsers drop them by spec).This appends
location.hashto the target, unless the stub'sredirect_toalready pins its own anchor. One shared layout — fixes every already-merged stub (#405/#406) and all upcoming retirements (#412/#413/#414) at once.No-JS readers keep the current behavior (meta refresh to the page top + the visible 'this page has moved' link) — that's the best the platform allows.
🤖 Generated with Claude Code