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.

Sign up for GitHub

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

Jump to bottom

Add ability to pass in an array of IValidator #177

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add ability to pass in an array of IValidator #177

wants to merge 1 commit into from

Conversation

ElliottBrand
Copy link

Added a parameter to pass in an array of IValidator, so validation can occur on all the provided validators.

@midoriiro
Copy link

midoriiro commented Apr 1, 2023 •
edited
Loading

How do you handle this case:
Validator is set with a validator of type IValidator<MyModel> and Validators is set with two validators for specific fields of MyModel ?

My guess is field/model validation in Validator branch will fail with Cannot validate instances of type 'MyFieldType'. This validator can only validate instances of type 'MyModel'..

Can you test this case ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ElliottBrand @midoriiro