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

SUSHI should replace ElementDefinition.type.aggregation array when modified in FSH #1411

Closed
cmoesel opened this issue Jan 16, 2024 · 0 comments · Fixed by #1433
Closed

SUSHI should replace ElementDefinition.type.aggregation array when modified in FSH #1411

cmoesel opened this issue Jan 16, 2024 · 0 comments · Fixed by #1433
Labels
bug Something isn't working

Comments

@cmoesel
Copy link
Member

cmoesel commented Jan 16, 2024

Some ElementDefinition array properties are additive (you should only add to them) while others should be fully replaced when edited (and perhaps there are other options too). I asked for more information on this when I filed FHIR-31405 back in 2021.

The ElementDefinition.type.aggregation array is an array property that should be replaced in the snapshot by whatever is in the differential. To support this, however, SUSHI needs to perform special handling of how it generates the differential. More background can be seen in this Zulip conversation.

Note that we will need to support things like this at a broader level, so any solution in FSH/SUSHI should be easy to apply to other element properties as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants