-
-
Notifications
You must be signed in to change notification settings - Fork 35
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: export SafeActionClient
type to make it portable
#267
fix: export SafeActionClient
type to make it portable
#267
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@brunorodmoreira is attempting to deploy a commit to the Edoardo Ranghieri's projects Team on Vercel. A member of the Team first needs to authorize it. |
Hi Bruno, thanks for your contribution. Does this solve #64? |
Yes! |
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.
👍
SafeActionClient
type to make it portable
🎉 This PR is included in version 7.9.3-beta.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Includes the `SafeActionClient` into the types export to make the inferred type of `createSafeActionClient` portable in order to avoid the error [TS2742](https://typescript.tv/errors/#ts2742). It enables declaring the action client in a shared package and import it in the other apps or packages. closes #64 Co-authored-by: Bruno Moreira <[email protected]>
@brunorodmoreira the fix indeed seems to work. It's just ridiculous to me that it was this easy all this time. Oh well, learned something today. Small but amazing contribution Bruno, thank you very much once again! Fix is included in v7.9.3. |
Includes the
SafeActionClient
into the types export to make the inferred type ofcreateSafeActionClient
portable in order to avoid the error TS2742. It enables declaring the action client in a shared package and import it in the other apps or packages.Proposed changes
Put your proposed changes here.
I've just exported the type in the
index.types.ts
file.Related issue(s) or discussion(s)