-
-
Notifications
You must be signed in to change notification settings - Fork 244
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
Improve Zod integration #1210
Comments
@radekmie can I add label feature? |
@RG7279805 Definitely! If you'd need help, feel free to file a draft PR first. |
@radekmie I have tried to add that label function in I have made a draft where I add a label in one problem tho label has to be passed everywhere in I am sorry if I did something wrong I hope you overlook it since it's my first commit in open source current draft : https://github.com/RG7279805/zod/tree/label-in-raw-params |
@RG7279805 I think we've misunderstood a little 😅 I meant the usage of it in uniforms, not adding a feature of Zod. In any case, I already saw you've raised it also in Zod's repo, so that's fine. |
@radekmie yeah it was my mistake 😃 |
I think that i opened an issue about Zod that could be integrated here #1292 |
Since we added Zod integration in #1169, there were a few open topics that we can finally address:
ZodErrorMap
and we should show an example of that in the docs and the playground..label(string)
function to a schema (Add a label function as like yup colinhacks/zod#1767) but in the meantime, we could use.describe(string)
..label
to get released, it should be a flag on the bridge whether the.description
should be used as a label.safeParse
throws a genericError
, source) we should have a flag on the bridge deciding it.The text was updated successfully, but these errors were encountered: