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

feat(validation-controller): revalidateErrors for object or property #461

Conversation

manuel-guilbault
Copy link
Contributor

This allows for what seems to me like a good enough fix for #279. It is easy to implement
a 'dependsOn' binding behavior calling this method to revalidate the errors of a specific
property.

@jdanyow
Copy link
Contributor

jdanyow commented Aug 12, 2017

nice 👍 - could you rename the arguments object and propertyName to keep things consistent with the rest of the APIs?

I understand why you probably didn't do that initially- to avoid changing the method too much but in this case I think it's important to keep the public APIs consistent.

Thanks!

@manuel-guilbault manuel-guilbault force-pushed the revalidateErrors-for-property branch from a2b7833 to 35c05ea Compare August 12, 2017 12:08
@manuel-guilbault
Copy link
Contributor Author

👍 done!

@EisenbergEffect
Copy link
Contributor

@jdanyow Can this be closed? It looks like it was merged already.

@jdanyow jdanyow closed this Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants