FirelyTeam / firely-validator-api

Firely's official FHIR validator API for validating HL7 FHIR resources against profiles.
Other
8 stars 2 forks source link

Extend ValidationResult class to allow localization/formatting #157

Open wmrutten opened 8 years ago

wmrutten commented 8 years ago

ValidationResult currently only exposes an ErrorMessage property. This prevents clients from localizing/formatting the error message. Proposal:

Advanced:

Maybe similar implementation to .NET exception classes?

brianpos commented 6 years ago

Yes this really needs to be done, and a thorough going through all the error messages to make them more user friendly, not just developer friendly. (Such as using the display property from the element definition instead of the path/slice name in the messages) and less intimidating words than instance, cardinality, etc

mmsmits commented 3 years ago

This issue is about dotnet attribute validation, not the profile validator, so I removed that label

ewoutkramer commented 3 years ago

So, the ValidationResult @wmrutten talks about is this one from the .NET framework: https://docs.microsoft.com/en-us/dotnet/api/system.componentmodel.dataannotations.validationresult?view=net-5.0

mmsmits commented 8 months ago

We have done this right? @ewoutkramer

ewoutkramer commented 8 months ago

@brianpos and I had a proposal, but I am not sure if it went anywhere.