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

Generating appointments along the side of Encounters #1362

Draft
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

tomkaith13
Copy link

@tomkaith13 tomkaith13 commented Sep 6, 2023

WIP PR just to share how Appointment resource can be generated along side of Encounters

@@ -24,13 +26,10 @@
import java.util.stream.Collectors;

import org.apache.commons.lang3.StringUtils;
import org.hl7.fhir.r4.model.Address;
import org.hl7.fhir.r4.model.AllergyIntolerance;
import org.hl7.fhir.r4.model.*;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the pull request!

We have not started a formal review yet, but I noticed the change to star-imports.

Please do not use star-imports -- they violate our style guidelines, and should appear as WARN messages when you run the unit tests (which incidentally are failing).

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @jawalonoski
Thanks for looking into it.

I just created the PR for others to share the code.
I wasnt intending to merge this 😅
Thats the reason I left it in draft.

I will try cleaning this up if you feel its worth pursuing.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this whole PR is a mess. 🤦
I ended up merging some Consent related changes in there as well which was me experimenting Consent generation per Patient which snuck into this PR.

I can close it out if it adds too much noise for you.

tomkaith13 and others added 18 commits September 8, 2023 10:20
 add consent generation for every encounter
add a custom extn to inject custom ids
added changes to create careplan tree based topology
adding Task generation which is tied to a CarePlan
…careplan

Added changes to reverse link ServiceRequest to the root-care-plan
…very-consent

add questionnaire-response and link them to consents
adding start and end times to appt resource
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 this pull request may close these issues.

2 participants