Skip to content
This repository was archived by the owner on Aug 7, 2020. It is now read-only.

Conversation

@ganeshkumar1989
Copy link
Contributor

@ganeshkumar1989 ganeshkumar1989 commented Apr 16, 2019

Trigger for validation in cases where the validation depends on other controls. This can be used, for example in confirm password field, where the filed depends on the actual password

Staging: https://ovh_ui_kit_documentation_new-feat_password_validation_trigger.uxci.ovh

@ganeshkumar1989 ganeshkumar1989 force-pushed the feat/password-validation-trigger branch 2 times, most recently from a4b3569 to c2a02cf Compare April 19, 2019 06:53
Copy link
Contributor

@AxelPeter AxelPeter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems better to use :)
Could you please add a unit test and remove the dist files from this commit ?
Thanks !

@ganeshkumar1989 ganeshkumar1989 force-pushed the feat/password-validation-trigger branch from c2a02cf to 07b9eb6 Compare April 22, 2019 06:29
trigger for validation in cases where the validation depends on other controls
@ganeshkumar1989 ganeshkumar1989 force-pushed the feat/password-validation-trigger branch from 07b9eb6 to 4efbac2 Compare April 22, 2019 06:32
@ganeshkumar1989
Copy link
Contributor Author

Seems better to use :)
Could you please add a unit test and remove the dist files from this commit ?
Thanks !

Thanks a lot @AxelPeter . The unit test cases have been added and the dist files have been removed from the commit.

@AxelPeter AxelPeter merged commit de19bd9 into develop Apr 23, 2019
@AxelPeter AxelPeter deleted the feat/password-validation-trigger branch April 23, 2019 08:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

3 participants