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

Nested validation? #937

Open
plettb opened this issue Sep 4, 2024 · 0 comments
Open

Nested validation? #937

plettb opened this issue Sep 4, 2024 · 0 comments
Labels
feature request 📬 A request for new changes to improve functionality

Comments

@plettb
Copy link

plettb commented Sep 4, 2024

Overview

The problem is explained quite clearly in #557. I believe there should be some kind of capability built into the toolkit to allow or facilitate one ObservableValidator to host another one, and when the parent "ValidateAllProperties()" is called, the call is passed through to the children. Whereas I can think of a few ways to implement this, I would bow to the wisdom of those closer to the project.

I haven't given tons of detail below, but I'd like to think it would be obvious. If not, I'll gladly provide more detailed information.

API breakdown

I don't think there are necessarily any API changed required.

Usage example

My preference would be that ValidateAllProperties() automatically (i.e. by default) also called the same method on child properties that implement "ObservableValidator".

Breaking change?

No

Alternatives

Whereas there is one possible work-around suggested in #557, it's not particularly elegant, and likely not the one I'd choose.

Additional context

No response

Help us help you

No, just wanted to propose this

@plettb plettb added the feature request 📬 A request for new changes to improve functionality label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request 📬 A request for new changes to improve functionality
Projects
None yet
Development

No branches or pull requests

1 participant