-
Notifications
You must be signed in to change notification settings - Fork 3
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
Support for dependent property validation #20
Comments
This may require something like a Anyway will chat with some of the others about |
As an option, I can use a fake rule on |
for now that may be a work around but it is a valid feature request, I just need to think about how best to handle it. |
There are some other quirks around this that need thinking about, from an intent perspective it is easier to have the notion of Also currently it seems to make sense to make this happen only on changes not on explicit validation, as if you are doing that you will be re-validating everything anyway so it seems pointless to trigger dependencies there as nothing could change once the validation starts (I think off top of my head). Finally at first pass it would seem that this should be something stored against a rule link, however as a property can have multiple rule links it would mean you would end up having to do the triggering at the rule link level not the property level, allowing people to trigger the same property multiple times etc. So it may actually need an entirely new notion which stores the property dependents at the property level not the rule level. Ultimately it needs a lot more thought around how it would all hook together, but I think we all know what we want it to do, just how best to make it happen. |
The use case is a registration form where a password and a confirmation are entered. When the password is changed the confirmation should also be validated to match the new password value
The text was updated successfully, but these errors were encountered: