rancher / dashboard

The Rancher UI
https://rancher.com
Apache License 2.0
450 stars 256 forks source link

Investigate using a standard form validation library #8365

Open nwmac opened 1 year ago

nwmac commented 1 year ago

Look at the 2 main Vue form validation libraries and see if using one of these would help.

Figure out how to wire in our existing validators to it, if so.

torchiaf commented 1 year ago

Candidates:

rak-phillip commented 1 year ago

Related Rancher Desktop discussion - rancher-sandbox/rancher-desktop#3376

cnotv commented 1 year ago

While evaluating alternatives to our form validation, we have to consider existing logic and edge cases which range from trivial to boring to very convoluted.

Out of my mind I can list these:

It would be also convenient to have the ability to validate component separately and map them to the form, so we may have the possibility to add validation directly to them and avoid repetition to each view, e.g. the case with the key duplication

cnotv commented 1 year ago

Other cases which may have been missed were mentioned in the documentation update issue https://github.com/rancher/dashboard/issues/7150

cnotv commented 1 year ago

Considerations about validation logic, which may eventually close this https://github.com/rancher/dashboard/issues/6159

nwmac commented 3 months ago

@gaktive Changing milestone to 2.10.0