docs(solid-query): replace the 'advanced-ssr' placeholder with a pointer to the SSR guide - #11588
Conversation
…ter to the SSR guide
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: TanStack/query/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe Advanced Server Rendering guide replaces its placeholder with a description of Solid Query hydration. It states that ChangesSolid SSR documentation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to The updated page directs readers to the SSR guide, which explains Solid Query hydration. No merge-blocking documentation issue is evident. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 f8c820e
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
size-limit report 📦
|
🎯 Changes
Replaces the
Will come soonplaceholder indocs/framework/solid/guides/advanced-ssr.mdwith a one-line pointer to the SSR guide.vue/svelte/preact/angular/litdon't have anadvanced-ssr.mdguide at all — onlyreactdoes, since it covers React Server Components and the Next.js app router, concepts specific to that ecosystem. Solid's placeholder traces back to #8950, which scaffolded the full set of react guide filenames for solid-query in bulk;advanced-ssr.mdwas copied along without content ever being written for it.I looked for solid-query-specific SSR material that could fill this page (checked
notifyManagerbatching, thereconcile/structuralSharingoption, and thesolid-start-streamingexample's/hydrationroute that compares query state across server/hydration/client). Everything either turned out not to be SSR-specific or was already covered by the SSR guide (#11581), so there's no separate "advanced" API surface to document — this page now says so and points to the guide that has it all.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit
useQueryhydrating automatically wherever it is used.