Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Include linked Patients recursively to the Patient Compartment #1297

Closed
alexanderkiel opened this issue Nov 23, 2023 · 1 comment
Closed
Assignees
Labels

Comments

@alexanderkiel
Copy link
Member

alexanderkiel commented Nov 23, 2023

According to the Patient Compartment Spec, all linked Patients are part of the original Patient Compartment. So we have to include them in blaze.ab.api/rev-include, so Patient $everything and other functionality will use them.

This issue is part of the Epic: #399

@alexanderkiel alexanderkiel added the bug Something isn't working label Nov 23, 2023
@alexanderkiel alexanderkiel added this to the v0.23.2 milestone Nov 23, 2023
@alexanderkiel alexanderkiel modified the milestones: v0.23.2, v0.23.3 Nov 26, 2023
alexanderkiel added a commit that referenced this issue Dec 5, 2023
We need the patient anyway in patient everything, but we added it later
after taking the max 10.0000 resources. Now in #1297 we will expand the
linked patients, so we will need to return the patients from
d/patient-everything because there could be more than the initial
patient.

I use the cat transducer and an extra eduction to concatenate the
patient-handle with the other resource handles.
@alexanderkiel
Copy link
Member Author

We decided not to follow links while generating the Patient compartment based on answers here: https://chat.fhir.org/#narrow/stream/179166-implementers/topic/Patient.20Components.20and.20Links

@alexanderkiel alexanderkiel removed this from the v0.23.3 milestone Jan 2, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants