Add timeout documentation example for QueriesHydration #1840
+118
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: manudeli <[email protected]>
Co-authored-by: manudeli <[email protected]>
Co-authored-by: manudeli <[email protected]>
Copilot
AI
changed the title
[WIP] Add timeout prop for QueriesHydration
Add timeout prop to QueriesHydration for server-side query time limits
Nov 21, 2025
Copilot finished work on behalf of
manudeli
November 21, 2025 07:39
Member
|
@copilot timeout should be implement by library user not prop with Could you remove new interface but add this example in QueriesHydration docs? |
People can be co-author:
|
…d timeout Co-authored-by: manudeli <[email protected]>
Contributor
Author
Removed the |
Member
|
@copilot rebase this branch, This paragraph also should be added in suspensive.org |
Co-authored-by: manudeli <[email protected]>
Contributor
Author
Added the timeout documentation to suspensive.org in both English and Korean. Commit: 55d2231 Note: I cannot rebase branches as force push is not allowed in my environment. If you need a rebase, please do it manually. |
Member
|
@copilot Ah,,, This shouldn't be worked...! do we have to make prop? |
|
Size Change: +144 B (+0.17%) Total Size: 86.8 kB
ℹ️ View Unchanged
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1840 +/- ##
=========================================
+ Coverage 0 93.38% +93.38%
=========================================
Files 0 44 +44
Lines 0 710 +710
Branches 0 177 +177
=========================================
+ Hits 0 663 +663
- Misses 0 41 +41
- Partials 0 6 +6
🚀 New features to boost your workflow:
|
Contributor
Author
Could you clarify what isn't working with the |
Copilot finished work on behalf of
manudeli
November 25, 2025 14:46
Member
|
@copilot How about this <QueriesHydration queries={timeoutQueries([userQueryOptions(), postsQueryOptions()], { ms: 2000 })}> ? |
Contributor
Author
Your comment appears to be incomplete - could you please share what you had in mind? I'd be happy to implement your suggestion once I understand it. |
Copilot finished work on behalf of
manudeli
November 25, 2025 15:07
Member
|
@copilot This is example what I want <QueriesHydration
queries={[userQueryOptions(), postsQueryOptions()].map((query) => timeoutOptions(query, { ms: 2000 }))}
>
...
</QueriesHydration> |
Contributor
Author
... The code block appears to be empty. Could you please share the example again? |
Copilot finished work on behalf of
manudeli
November 25, 2025 15:17
Member
|
@copilot This is example what I want <QueriesHydration
queries={[userQueryOptions(), postsQueryOptions()].map((query) => timeoutOptions(query, { ms: 2000 }))}
>
</QueriesHydration> |
Contributor
Author
... The code block is still showing as empty. This might be a GitHub formatting issue. Could you try sharing the code in a different way, perhaps as a gist link or by using different formatting? |
Copilot finished work on behalf of
manudeli
November 25, 2025 15:23
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Overview
Adds documentation example showing how to implement timeout for
QueriesHydrationcomponent in@suspensive/react-query-4and@suspensive/react-query-5using user-implementedqueryOptions.Based on maintainer feedback, timeout functionality should be implemented by library users via
queryOptionsrather than as a built-in prop. This PR adds documentation examples demonstrating the recommended pattern in both JSDoc and suspensive.org documentation.Note: Awaiting maintainer feedback on the final implementation approach. The maintainer's code examples are appearing as empty code blocks - clarification has been requested.
Usage Example
Queries exceeding the timeout will trigger an error and fall back to client-side rendering (default
skipSsrOnErrorbehavior) or proceed based on configuration.Changes Made
QueriesHydrationcomponent in@suspensive/react-query-4and@suspensive/react-query-5PR Checklist
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.