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
TextSeparator = ", " for concatenating errors #2925
Labels
Comments
It seems like something someone could set, and perhaps the default should be |
Freelancingonupwork
added a commit
to Freelancingonupwork/csla
that referenced
this issue
May 15, 2024
…ing-errors/main
rockfordlhotka
added a commit
to Freelancingonupwork/csla
that referenced
this issue
May 17, 2024
…ing-errors/main
rockfordlhotka
added a commit
to Freelancingonupwork/csla
that referenced
this issue
May 20, 2024
…ing-errors/main
rockfordlhotka
added a commit
that referenced
this issue
May 20, 2024
* Add DisplayIndex Property to Business Rules * Test * Update * Text separator for concatenating errors * Chnages --------- Co-authored-by: Stefan Ossendorf <StefanOssendorf@users.noreply.github.com> Co-authored-by: Mayur Kasidariya <mayur.kasidariya@potenzaglobalsolutions.com> Co-authored-by: Rockford Lhotka <rocky@lhotka.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The standard validation error messages such as
[Required]
are proper sentences ending with a.
and I think this is good practice.Csla.Blazor.PropertyInfo
assumes error messages won't end with a.
using", "
to concatenate errors, so you end up with:Should
TextSeparator = " "
or perhaps be apublic static string TextSeparator { get; set; }
instead?The text was updated successfully, but these errors were encountered: