docs(angular): fix QueryClient field in paginated guide#10147
docs(angular): fix QueryClient field in paginated guide#10147pavan-sh wants to merge 1 commit intoTanStack:mainfrom
Conversation
|
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughA public class field in an Angular example component is converted to a private field using private field syntax, changing Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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 159d1dc
☁️ Nx Cloud last updated this comment at |
🎯 Changes
this.#queryClient... (with #), which in TS/JS means a private class field. The declaration queryClient = inject(QueryClient) creates a public field named queryClient, so this.#queryClient would be undefined and the snippet breaks.
So I changed the declaration to match what the code is actually calling:
• from: queryClient = inject(QueryClient)
• to: #queryClient = inject(QueryClient)
✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit