Accept observable in mergeValidators/setValidators #93
Unanswered
rafaelss95
asked this question in
Ideas
Replies: 1 comment
-
|
You're welcome to create a PR. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm submitting a...
Current behavior
Currently
mergeValidators/setValidatorsdoesn't accept observables... we have thevalidateOnthat deals with custom errors, but not with validators.Expected behavior
I'd expect to see
mergeValidators/setValidatorsaccepting observables also, assetValue/patchValue.What is the motivation / use case for changing the behavior?
I have an use case where I set the validators based on another
formControlvalue, e.g.:Also, I think we could add support to accept observable in
mergeErrors/setErrors, that would remove the necessity of havingvalidateOn... but that's just an idea.Environment
Beta Was this translation helpful? Give feedback.
All reactions