-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(vue-query): ensure built-in types are not being unwrapped #5954
fix(vue-query): ensure built-in types are not being unwrapped #5954
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 46051ed. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 2 targetsSent with 💌 from NxCloud. |
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 30766a5:
|
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 46051ed:
|
Codecov ReportPatch has no changes to coverable lines. ❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
📢 Thoughts on this report? Let us know!. |
what about user-defined classes? thanks for picking this up so quickly! |
When dealing with user-defined classes, we can draw inspiration from Vue's handling method, as demonstrated here: A possible implementation might resemble the following: declare module '@tanstack/vue-query' {
export interface RefUnwrapBailTypes {
UserDefinedClass: UserDefinedClass;
}
} However, it's important to note that @tanstack/query-core will convert the Given this consideration, I'm not sure if It would be great to get feedback from @DamianOsipiuk . |
Resolve the following error
This error was mentioned in PR #5935 (comment)
In that PR, not avoiding unwrapping built-in types resulted in errors, such as Ref, occurring.
Before
After