fix(redirects): redirect stale /questions/topic/replay-vision to /replay-vision - #19808
Open
posthog[bot] wants to merge 2 commits into
Open
fix(redirects): redirect stale /questions/topic/replay-vision to /replay-vision#19808posthog[bot] wants to merge 2 commits into
posthog[bot] wants to merge 2 commits into
Conversation
…lay-vision The old /questions/topic/replay-vision route returns 404 for direct visits and for Gatsby page-data prefetches. The replacement page lives at /replay-vision. Add a redirect so both the direct URL and the prefetch resolve. Generated-By: PostHog Desktop Task-Id: 9877db9c-7e3e-481d-9e91-9352b7f3bf2a
Contributor
Author
🦔 PostHog Review reviewed this pull requestFound 0 must fix, 1 should fix, 0 consider. Published 1 finding (view the review). Resolved comments: 1 fixed |
Contributor
Deploy preview
|
Contributor
Author
|
PostHog Review alpha 🦔 If you find any issues helpful - please reply "valid", "invalid", etc., for evaluation purposes 🙏 |
![posthog[bot]](https://avatars.githubusercontent.com/in/1203049?s=60&v=4){kind=small}
Contributor
Bundle reportTotal JS (gzip)8.15 MiB (+0.0 KiB / +0.0%) Eager graph (modules shipped in each entrypoint's initial chunks)
Largest modules in the
|
| Module | Size |
|---|---|
./src/data/mcp-tools.json |
1053.9 KiB |
css ./node_modules/.pnpm/css-loader@5.2.7_webpack@5.101.3/node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[1]!./node_modules/.pnpm/postcss-loader@4.3.0_postcss@8.5.6_webpack@5.101.3/node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[2]!./src/styles/global.css |
754.5 KiB |
./src/components/Stickers/Stickers.tsx |
696.4 KiB |
./node_modules/.pnpm/@radix-ui+react-icons@1.3.2_react@18.3.1/node_modules/@radix-ui/react-icons/dist/react-icons.esm.js |
481.4 KiB |
./node_modules/.pnpm/rehype-raw@7.0.0/node_modules/rehype-raw/lib/index.js + 29 modules |
395.1 KiB |
./src/hooks/useCustomers.tsx + 55 modules |
370.0 KiB |
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js |
354.8 KiB |
./node_modules/.pnpm/react-markdown@8.0.7_@types+react@16.14.66_react@18.3.1/node_modules/react-markdown/lib/react-markdown.js + 88 modules |
351.4 KiB |
./src/components/ProductComparisonTable/index.tsx + 126 modules |
301.7 KiB |
./node_modules/.pnpm/cloudinary-core@2.14.0_lodash@4.17.21/node_modules/cloudinary-core/cloudinary-core.js |
281.9 KiB |
./src/components/SearchUI/index.tsx + 87 modules |
273.0 KiB |
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/magnifying-glass.mjs |
254.7 KiB |
./node_modules/.pnpm/framer-motion@10.18.0_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/framer-motion/dist/es/render/dom/motion.mjs + 109 modules |
253.9 KiB |
./node_modules/.pnpm/d3@7.9.0/node_modules/d3/src/index.js + 208 modules |
247.4 KiB |
./src/components/Pricing/PricingSlider/Slider.tsx + 87 modules |
240.1 KiB |
Eager-graph budgets are report-only until a baseline is established. Sizes are gzip of public/**/*.js; eager size is webpack module source bytes for the modules actually shipped in the entrypoint's initial chunks (post-tree-shake).
… topic The Replay Vision page's community section built its forum URL from the product slug (/questions/topic/replay-vision), but that topic page does not exist — forum topic 377 is shared with Session Replay and lives at /questions/topic/session-replay. With the new redirect in place, all three community links (search forums, "be the first to ask", view all questions) resolved back to /replay-vision itself, and Gatsby kept prefetching the non-existent page-data URL. Add an optional forumTopicSlug to the product data (defaulting to the slug) and set it to Session Replay's slug for Replay Vision, so the links resolve directly to the real topic and the stale prefetch stops. Generated-By: PostHog Desktop Task-Id: ab574683-c9e3-4695-a8d3-276208d2bcfc
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.
Changes
/questions/topic/replay-visionroute resolves to/replay-visioninstead of returning 404./page-data/questions/topic/replay-vision/page-data.jsonfailed silently./replay-visionreturns HTTP 200./questions/topic/*redirect convention (a Vercel default 308 permanent redirect).Why: A stale topic route left over from the move to the
/replay-visionmarketing page caused steady website friction — visible 404s for external visitors and prefetch misses for internal navigation.Checklist
vercel.jsonCreated with PostHog Desktop from this inbox report.