-
Notifications
You must be signed in to change notification settings - Fork 659
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
base: master
Are you sure you want to change the base?
Conversation
@@ -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.*; |
There was a problem hiding this comment.
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).
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
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
WIP PR just to share how Appointment resource can be generated along side of Encounters