Skip to content

Include PropertyName in MemberNames for ValidationResult in FluentVal… #2744

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

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

Conversation

tdy-niek-klaverstijn
Copy link

…idator

GitHub Issue (If applicable): closes #2741

PR Type

  • Feature

What is the current behavior?

Only the message is passed to the validation result, discrimination based on property name is not possible when consuming it.

What is the new behavior?

Property name is passed to the member names array of the ValidationResult

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • [x ] Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

@kazo0
Copy link
Contributor

kazo0 commented Apr 30, 2025

@tdy-niek-klaverstijn thanks for the submission!

I see the checks are failing for Conventional Commits, see https://www.conventionalcommits.org/ for help on how to structure the commit messages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pass PropertyName to MemberNames on ValidationResult in FluentValidator
2 participants