feat: improve grid validation #508
Conversation
|
Task linked: DEV-11849 Improve config validation for properties |
Deployed with Cloudflare Pages ☁️ 🚀 🆗 |
| } catch (error: unknown) { | ||
| ctx.addIssue({ | ||
| code: z.ZodIssueCode.custom, | ||
| message: formatMessage('errors_invalid_input', { | ||
| name: capitalize(type), | ||
| }), | ||
| path: ['src'], | ||
| }) |
There was a problem hiding this comment.
Why don't we use error object in the addIssue? It should have the information about which field is invalid.
There was a problem hiding this comment.
By error object you mean ctx instance? I've checked but it's empty in this case:
There was a problem hiding this comment.
No, the error: unknown.
It should return the information about the invalid field if such error is thrown.
There was a problem hiding this comment.
The error we catch here comes from parsePlatformInput, it doesn't have anything to do with zod unfortunatelly :/ I guess zod has a problem here to provide exact path because we run transform over whole object, it's not single field transform where path is easy to read.
| import { computedAsync } from '@vueuse/core' | ||
| import { ZodEffects, ZodError, type ZodObject } from 'zod' | ||
|
|
||
| export const useForm = ( |
There was a problem hiding this comment.
I actually once saw you using a form somewhere and was going to suggest something like useForm for vue but it seems you are implementing your own here
Ticket ID
DEV-11849
DEV-11908
DEV-11916
Description