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.

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

Improve error messages for slices #224

Closed
ewoutkramer opened this issue Dec 11, 2023 · 5 comments · Fixed by #231
Closed

Improve error messages for slices #224

ewoutkramer opened this issue Dec 11, 2023 · 5 comments · Fixed by #231
Assignees

Comments

@ewoutkramer
Copy link
Member

ewoutkramer commented Dec 11, 2023

Here, the name of the element (Observation.value) is somewhere in the middle and repeated completely at the end.

[ERROR] Element is of type 'boolean', which is not one of the allowed choice types ('string','Quantity') (at [0].entry[2].resource[0].valueBoolean[0], element Bundle(http://example.org/StructureDefinition/DocumentBundle).entry.resource->Observation.value[x][@default])

Is this good enough? Just make an exception for slices? Everywhere?

@mmsmits mmsmits self-assigned this Dec 14, 2023
@mmsmits
Copy link
Member

mmsmits commented Dec 14, 2023

I'll split this into two separate issues

@mmsmits mmsmits changed the title Improve error messages Improve error messages for slices Dec 14, 2023
@mmsmits
Copy link
Member

mmsmits commented Dec 14, 2023

Moved the other issue here:
#232

@ewoutkramer
Copy link
Member Author

This is actually the "open" issue that you split up ;-)

@ewoutkramer ewoutkramer reopened this Dec 14, 2023
@mmsmits
Copy link
Member

mmsmits commented Dec 14, 2023

ah oops

@ewoutkramer
Copy link
Member Author

Wait, it all got mixed up. I'll close this one and update the other. So this one was about "slices".

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 a pull request may close this issue.

2 participants