Uh oh!
There was an error while loading. Please reload this page.
docs(svelte): replace deprecated isLoading with isPending in overview - #10990
Conversation
`isLoading` is a deprecated alias for `isPending && isFetching` and is scheduled for removal in the next major version. The overview example shows a standard (always-enabled) query, so `isPending` is the correct flag. All other framework overview docs (React, Vue, Angular, Lit, Preact) already use `isPending`. Note: `isLoading` is intentionally retained in disabling-queries guides where it is the right flag for disabled/lazy queries. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Svelte overview example changes the loading-state condition in the code sample from ChangesSvelte overview example
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 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 |
Uh oh!
There was an error while loading. Please reload this page.
isLoadingis a deprecated alias forisPending && isFetchingand is scheduled for removal in the next major version. The overview example shows a standard (always-enabled) query, soisPendingis the correct flag. All other framework overview docs (React, Vue, Angular, Lit, Preact) already useisPending.Note:
isLoadingis intentionally retained in disabling-queries guides where it is the right flag for disabled/lazy queries.🎯 Changes
✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit