-
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
Draft
tomkaith13
wants to merge
28
commits into
synthetichealth:master
Choose a base branch
from
tomkaith13:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+453
−63
Draft
Changes from 10 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
945d300
added .idea to gitignore
tomkaith13 76e05f3
added a dummy appt resource
tomkaith13 46f5bc0
added changes to report Appointments based on Encounters
tomkaith13 4b27b09
fixed some appointment related execute bundle errors
tomkaith13 7207bc2
add patient in appt
tomkaith13 3114575
added identifier to the appt
tomkaith13 3d75c6e
added custom extension stringValue
tomkaith13 ee9bee3
removed unused variable
tomkaith13 e94889a
add valid attribs for Consent
tomkaith13 4dc6d46
Merge pull request #1 from tomkaith13/consent
tomkaith13 9dc7bcb
add consent generation for every encounter
tomkaith13 33dfe3f
Changed the code to a sha256 hex string for better searchability
tomkaith13 39f1045
cleaned up a few imports
tomkaith13 6713fe5
Merge pull request #2 from tomkaith13/one-time-consents
tomkaith13 02e1952
add a custom extn to inject custom ids
tomkaith13 01acac6
add multiple extension to consent
tomkaith13 0f2a6e9
Merge pull request #3 from tomkaith13/consent-with-custom-extn
tomkaith13 f97815c
added changes to create careplan tree based topology
tomkaith13 93cce14
Merge pull request #4 from tomkaith13/add-children-careplans
tomkaith13 d378810
adding Task generation which is tied to a CarePlan
tomkaith13 878ba9a
Merge pull request #5 from tomkaith13/add-task-as-revlink-to-careplan
tomkaith13 8cb93fa
Added changes to reverse link ServiceRequest to the root-care-plan
tomkaith13 86e8a94
Merge pull request #6 from tomkaith13/add-service-request-revlink-to-…
tomkaith13 5ec445d
add questionnaire-response and link them to consents
tomkaith13 6e3e4c4
add comments and remv extra consent ref link
tomkaith13 f67209f
Merge pull request #7 from tomkaith13/add-questionnaire-response-to-e…
tomkaith13 fac1816
adding start and end times to appt resource
tomkaith13 8149864
Merge pull request #8 from tomkaith13/add-appt-start-and-end
tomkaith13 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,9 +9,10 @@ bin/** | |
|
||
/.gradle/ | ||
/bin/ | ||
|
||
/.idea/ | ||
/failed_exports/ | ||
/output/ | ||
|
||
/out/ | ||
|
||
**/*.log | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.