TanStack
Update HydrationStreamProvider.tsx + Advanced SSR docs to fix unnecessary suspensions
This PR is based on my findings in #6116 and fixes two cases of unnecessary suspensions during streaming SSR, one on code and the other in the advanced SSR docs. See the full writeup in this comment, which should have all the info needed for reviewing this PR.
The latest updates on your projects. Learn more about Vercel for Git βοΈ
1 Ignored Deployment
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| query | β¬οΈ Ignored (Inspect) | Visit Preview | Feb 18, 2024 8:25am |
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
This pull request is automatically built and testable in CodeSandbox.
To see build info of the built libraries, click here or the icon next to each commit SHA.
Latest deployment of this branch, based on commit e8955373c3d0f0ef5062bb325be4ca57c931d0d8:
| Sandbox | Source |
|---|---|
| @tanstack/query-example-angular-basic | Configuration |
| @tanstack/query-example-react-basic-typescript | Configuration |
| @tanstack/query-example-solid-basic-typescript | Configuration |
| @tanstack/query-example-svelte-basic | Configuration |
| @tanstack/query-example-vue-basic | Configuration |
![codesandbox-ci[bot] avatar](https://avatars.githubusercontent.com/in/38409?v=4 "Published by a pub.dev verified publisher"){kind=small}
βοΈ Nx Cloud Report
CI is running/has finished running commands for commit e8955373c3d0f0ef5062bb325be4ca57c931d0d8. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
π See all runs for this CI Pipeline Execution
β Successfully ran 1 target
Sent with π from NxCloud.
![nx-cloud[bot] avatar](https://avatars.githubusercontent.com/in/80458?v=4 "Published by a pub.dev verified publisher"){kind=small}
@juliusmarminge would you mind taking a look here? Also I think the comment in the issue is super interesting:
- https://github.com/TanStack/query/issues/6116#issuecomment-1904051005
Codecov Report
Merging #6753 (c2d404b) into main (61f4d74) will not change coverage. Report is 1 commits behind head on main. The diff coverage is
n/a.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## main #6753 +/- ##
=======================================
Coverage 41.78% 41.78%
=======================================
Files 178 178
Lines 7014 7014
Branches 1416 1416
=======================================
Hits 2931 2931
Misses 3721 3721
Partials 362 362
