nucleuscloud / neosync

Open source data anonymization and synthetic data orchestration for developers. Create high fidelity synthetic data and sync it across your environments.
https://www.neosync.dev
Other
3.45k stars 124 forks source link

[NEOS-1305] Merge JS validate on type with validation button #2419

Open nickzelei opened 3 months ago

nickzelei commented 3 months ago

The validation button uses a separate validation state even though we currently have validation at the Yup layer. We should remove the separate validation state and just use the form state since we can rely on that to tell us if the field is valid or not.

The other alternative is to not validate the state at the Yup form level and just keep the separate state.

Either way the experience is weird right now because we are actively validating the input, but then I as a user must click validate in order to submit….

From SyncLinear.com | NEOS-1305

nickzelei commented 3 months ago

I see why we need both, but would be great if they fed into one another somehow..