You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently there is support for required and partial support for pattern (is there a reason why its supported only in query parameters ?).
I think that for the node client it can be very convenient right away.
The angular client is more complex because in order to provide a good user experience its need to be through the form validation mechanism.
Anyway, I'm using the angular client and I'll try to think what is the best way to use one validation mechanism (based on the swagger spec) while still embracing angular forms -> I'm trying to use the swagger as one source of truth whenever I can without making manual syncing 😉
The amazing swagger-tools project already implemented this kind of validation but as a middleware for the server side. I submitted this issue in order to maybe taking advantage of that.
I think that support for that will make this project even more awesome :)
@wcandillon Is it something that you think should be part of this project ?
Boris :)
The text was updated successfully, but these errors were encountered:
Currently there is support for required and partial support for pattern (is there a reason why its supported only in query parameters ?).
I think that for the node client it can be very convenient right away.
The angular client is more complex because in order to provide a good user experience its need to be through the form validation mechanism.
Anyway, I'm using the angular client and I'll try to think what is the best way to use one validation mechanism (based on the swagger spec) while still embracing angular forms -> I'm trying to use the swagger as one source of truth whenever I can without making manual syncing 😉
The amazing swagger-tools project already implemented this kind of validation but as a middleware for the server side. I submitted this issue in order to maybe taking advantage of that.
I think that support for that will make this project even more awesome :)
@wcandillon Is it something that you think should be part of this project ?
Boris :)
The text was updated successfully, but these errors were encountered: