Skip to content

Compare validator doesn't show validation error #15915

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

Closed
sarathlalsaseendran opened this issue Nov 29, 2019 — with docs.microsoft.com · 3 comments · Fixed by #15876
Closed

Compare validator doesn't show validation error #15915

sarathlalsaseendran opened this issue Nov 29, 2019 — with docs.microsoft.com · 3 comments · Fixed by #15876
Assignees
Labels
Blazor doc-enhancement Pri1 Source - Docs.ms Docs Customer feedback via GitHub Issue

Comments

Copy link

Hi, I have checked with Compare validator in viewmodel for Email field. It is not showing any error in validation summary. Can you please check that?


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@dotnet-bot dotnet-bot added ⌚ Not Triaged Blazor Source - Docs.ms Docs Customer feedback via GitHub Issue labels Nov 29, 2019
@guardrex
Copy link
Collaborator

Hello @sarathlalsaseendran ... It looks like this is a 🐞 bug that will be fixed for 3.1.

dotnet/aspnetcore#10643

I'll take a look at this topic next week to see about placing a 3.0-only note to cover this for 3.0. Thanks for writing in.

@guardrex
Copy link
Collaborator

guardrex commented Nov 29, 2019

If you try 3.1 and it isn't fixed, open a new issue on the engineering repo for them to look at it (go with a new issue because we don't work well on closed issues). Show them the exact code that you're using. If you want, you can add a "cc: @guardrex" to the bottom of your issue over there so that I can track it. Otherwise, open a new issue here on this topic after you find out what they say if this topic requires more work.

For this issue and the bug fix that they've put in, I'll ping you on the PR next week. Leave this open ... it will automatically close when the PR merges.

Copy link
Author

Hello @guardrex, thank you very much for the quick reply. I will check with 3.1 and update you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blazor doc-enhancement Pri1 Source - Docs.ms Docs Customer feedback via GitHub Issue
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants