docs(vue-query/guides/ssr): fix 'queryKey' to use array form - #11202
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan includes up to 10 reviews per rolling hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe Nuxt 2 SSR example now uses an array query key for the second ChangesNuxt 2 SSR example
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a localized documentation correction with no published-code or runtime impact, and no actionable merge-blocking risk remains beyond normal checks and review. 🚥 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 7a84f6a
☁️ 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
Fixes
queryKey: 'todos2'(a string) toqueryKey: ['todos2'](an array) in the seconduseQueryexample in the Nuxt 2 SSR guide.QueryKeyis typed asReadonlyArray<unknown>(packages/query-core/src/types.ts), and the firstuseQuerycall in the same snippet correctly usesqueryKey: ['todos'].This traces back to the v5 migration of this guide (#8001), where the original v4 call
useQuery("todos2", getTodos)— v4 allowed a bare string key — was converted to v5's object form but the key wasn't wrapped in an array.✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit