docs(react): fix broken anchors in SSR and migration guides#10214
docs(react): fix broken anchors in SSR and migration guides#10214TkDodo merged 2 commits intoTanStack:mainfrom
Conversation
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughCorrected cross-reference anchors and wording in two React Query docs: updated an anchor in the migration guide and clarified removal of the old cancel method; in the SSR guide fixed link anchors, added a cross-reference to deeper hydration/prefetching examples, and updated a guide link. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 211cd59
☁️ Nx Cloud last updated this comment at |
🎯 Changes
Fix broken internal anchor links in React docs.
While running the docs link checker, I found a few references pointing to anchors that don’t exist anymore (likely due to heading changes). This PR updates them to valid anchors and removes one stale anchor reference in the migration guide.
Changes included:
ssr.md: fix the “Dependent Queries & Code Splitting” anchor to match the actual heading slug (#dependent-queries-code-splitting)ssr.md: add a direct link to that section near the top for easier discoverymigrating-to-react-query-4.md: fix the disabling-queries anchor and remove the stale#old-cancel-functionanchor reference (link now points to the page, since that heading doesn’t exist)✅ Checklist
🚀 Release Impact
🧪 Test plan
pnpm installpnpm test:docsSummary by CodeRabbit