Skip to content

Conversation

@karrui
Copy link
Collaborator

@karrui karrui commented Nov 14, 2025

TL;DR

Export the getQueryClient function and update its documentation to clarify usage guidelines.

What changed?

  • Added the export keyword to the getQueryClient function to make it accessible outside the module
  • Rewrote the documentation comment to provide clearer guidance on when and how to use this function
  • Simplified the warning message while maintaining the emphasis on proper error handling

How to test?

  1. Import getQueryClient from the module in another file
  2. Verify that the function can be accessed and used correctly
  3. Check that the documentation appears correctly in IDE tooltips/intellisense

Why make this change?

This change makes the query client accessible to other parts of the application while providing clearer guidance on its proper usage. The updated documentation better explains the specific use case for this function and emphasizes the importance of error handling when using it in server components.

@vercel
Copy link

vercel bot commented Nov 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
starter-kit Ready Ready Preview Comment Nov 14, 2025 9:49am

Copy link
Collaborator Author

karrui commented Nov 14, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@karrui karrui changed the title feat: export getQueryClient and update comment on usage feat(web): export getQueryClient and update comment on usage Nov 14, 2025
@karrui karrui marked this pull request as ready for review November 14, 2025 09:49
@karrui karrui merged commit b5555e1 into main Nov 17, 2025
14 checks passed
@karrui karrui deleted the 11-14-feat_export_getqueryclient_and_update_comment_on_usage branch November 17, 2025 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant