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
/setValidators
doesn't accept observables... we have thevalidateOn
that deals with custom errors, but not with validators.Expected behavior
I'd expect to see
mergeValidators
/setValidators
accepting 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
formControl
value, 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