Fix skip
causing loading
to always be true
during SSR
#7567
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR #7310 introduced a regression that causes skipped SSR queries to always set
loading
totrue
, which doesn't match the behavior ofuseQuery()
on the client and can result in hydration mismatches.The problem is that
skip: true
was being treated as equivalent tossr: false
, but they're not actually equivalent. I think the correct solution is to letthis.getQueryResult()
provide a suitable result whenskip
is truthy, which ensures that skipped SSR queries will get the same result as skipped non-SSR queries.An alternative approach would be to simply back out the changes from PR #7310, but then we'd lose the benefit of avoiding the unnecessary
QueryPromise
whenskip
is truthy.Fixes #7380 (note that this issue's title and description are inaccurate, but its comments describe the problem correctly)
Checklist: