-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
chore: update .connect to .mutate #4686
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Ideally we could extend this in the following ways:
- Remove
'mutate' | 'mutateAsync'
from being omitted from the query return type (React, Vue) - Add
@deprecated
comments to theXMutate
andXMutateAsync
type helpers (e.g.ConnectMutate
andConnectMutateAsync
) - Add
mutate
/mutateAsync
for all mutation hooks (along with the above and@deprecated
comments for named return methods)
You are welcome to work on this or I can merge you PR into another branch and do this when I get some time.
useConnect()
:.connect
and.connectAsync
,.mutate
and.mutateAsync
,mutate
,connect
tomutate
in docs for Vue and ReactuseDisconnect()
:.disconnect
and.disconnectAsync
,.mutate
and.mutateAsync
,mutate
,disconnect
tomutate
in docs for Vue and React