Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REQUEST] Add option for only manual validation of a combined field #71

Open
omer-g opened this issue Mar 4, 2022 · 2 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@omer-g
Copy link
Contributor

omer-g commented Mar 4, 2022

Is your feature request related to a problem? Please describe.
I have two fields that get validated only on:blur. There is a combined field comprised of the other two, that performs additional validation of the two fields. Since the combined field validators are called whenever there is any change in the two fields, it is as if the two are checked on:change, which results unwanted and unnecessary validations (and many API requests).

Describe the solution you'd like
Would be great to have an option like manualValidation in combined fields to only validate manually. So when that option is enabled, validation will occur only when explicitly called. As a default it should probably be false.

Additional context
If this is implemented, perhaps it should come with a valid option for combined fields, so one could set valid to false and with that make sure that manual validation must be called for the combined field.

@omer-g omer-g added the enhancement New feature or request label Mar 4, 2022
@chainlist
Copy link
Owner

Hi again @omer-g,

Private life's been a bit busy recently.
If you are willing to make a change and create a pull request it would be cool! otherwise i'm going to work on it as soon as I can.

@omer-g
Copy link
Contributor Author

omer-g commented Mar 20, 2022

No worries, thank you for your response. I don't think I will get to making the change myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants