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

Validation messages are link to the first child object when using a list in the Model of the edit context #36

Open
yveslaurentcreton opened this issue Aug 24, 2020 · 5 comments
Open

Validation messages are link to the first child object when using a list in the Model of the edit context #36

yveslaurentcreton opened this issue Aug 24, 2020 · 5 comments

Comments

@yveslaurentcreton
Copy link

When using a list with multiple child object in the model of the edit context,
all the validation messages of all the child objects are linked to the first child object.
@ryanelian
Copy link
Owner

Working as intended. If requiring a different / more advanced behavior, PR welcome.

@yveslaurentcreton yveslaurentcreton mentioned this issue Nov 13, 2020
Merged
@ryanelian
Copy link
Owner

ryanelian commented Nov 13, 2020 •
edited
Loading

I need a more detailed information about this issue. For example, a repro and screenshots.

@ryanelian ryanelian reopened this Nov 13, 2020
@yveslaurentcreton
Copy link
Author

Ok, I'll try to give you a description or project to reproduce this issue, I'll keep you informed when I have it. Thank you for taking a look at it.

@yveslaurentcreton
Copy link
Author

FluentValidation.Blazor.Repro.zip

I've created a repro project. The example is on the home page of the repro project. Just click the submit button and then you can see that all the validation messages appear at the first address instead of at their own address.
The suggested fix from PR #37 fixes this issue.

ryanelian added a commit that referenced this issue Nov 25, 2020
@ryanelian
Merge pull request #37 from BEzGumpTion/master
936978c 
#36 Added arrayindex of the object to modelObjectPath in ModelGraphCache
@sergshu
Copy link

sergshu commented Jun 16, 2021

Is this solved in 3.0 v?

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

No branches or pull requests
3 participants
@sergshu @ryanelian @yveslaurentcreton