-
Notifications
You must be signed in to change notification settings - Fork 1
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: consent screen use mutate over async mutate #482
Conversation
src/hooks/useConsent.ts
Outdated
options: UseMutationOptions< | ||
PatchConsentDirectives['Response'], | ||
unknown, | ||
PatchConsentDirectives['Request'] & { | ||
directiveId: string; | ||
} | ||
> = {}, |
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.
Rather than provide the full options here, can we just provide an onSuccess
option? That will simplify the typing a lot, and make a soon-to-come change from me a lot easier.
e4357d3
to
394b576
Compare
Pull Request Test Coverage Report for Build 7063285931Warning: This coverage report may be inaccurate.We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
💛 - Coveralls |
🎉 This PR is included in version 9.1.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
mutate
overmutateAsync
, makes using it a bit simpler especially in the custom screen flowScreenshots