fix(examples/integrations): replace deprecated 'isServer' with 'environmentManager.isServer()' in the Next.js examples#10858
Conversation
…onmentManager.isServer()' in the Next.js examples
|
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)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughMigrate server detection from deprecated ChangesServer Detection Refactoring
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 5fe2829
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview3 package(s) bumped directly, 22 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
🎯 Changes
Migrate the remaining
isServerusages in the Next.js examples and integrations toenvironmentManager.isServer(), matching the migration already done forreact-query/preact-query(#10199),vue-query(#10826), andreact-query-next-experimental(#10857).Updated files:
integrations/react-next-15/app/providers.tsxintegrations/react-next-16/app/providers.tsxexamples/react/nextjs-suspense-streaming/src/app/page.tsxWithout these updates, users copy-pasting the Next.js provider snippet from the integration examples would import the deprecated
isServerexport and see the@deprecatedwarning in their IDE.✅ Checklist
🚀 Release Impact
Summary by CodeRabbit