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

Updates to accommodate data displays in EPIC #20

Draft
wants to merge 54 commits into
base: main
Choose a base branch
from

Conversation

achen2401
Copy link

@achen2401 achen2401 commented Oct 18, 2023

https://www.pivotaltracker.com/story/show/186276844
https://www.pivotaltracker.com/story/show/186277060

Changes include:

  • allows queries for EPIC data via config, REACT_APP_EPIC_QUERIES
  • update code to accommodate FHIR Questionnaire resource that contains no items (as in EPIC)
  • add new FHIR util functions
    screenshot from testing using sample EPIC questionnaire and questionnaire responses JSONs:
    Screen Shot 2023-10-18 at 1 03 16 PM

example screen shoot for the new observation/smart data section:
Screen Shot 2023-10-18 at 1 03 01 PM

@mcjustin
Copy link
Member

Thanks for the major effort @achen2401 !

If I'm reading this right, the observation/smart data section is labelled "Procedures / Diagnoses"? I think it should be "Observations" for now.

Bill & Lydia might want to wordsmith it later, or re-organize the sections, as the existing "Pertinent Medical History" label for Conditions implies a superset that might also include Observations. But, after you make the change above can you ask Ivan or Paul to bring this branch to the dev system after you make the above change, and we can see what it looks like with a variety of Epic data? That will inform these decisions.

Reference that I'm looking at: screenshot from September here.

@achen2401
Copy link
Author

Thanks for the major effort @achen2401 !

If I'm reading this right, the observation/smart data section is labelled "Procedures / Diagnoses"? I think it should be "Observations" for now.

Bill & Lydia might want to wordsmith it later, or re-organize the sections, as the existing "Pertinent Medical History" label for Conditions implies a superset that might also include Observations. But, after you make the change above can you ask Ivan or Paul to bring this branch to the dev system after you make the above change, and we can see what it looks like with a variety of Epic data? That will inform these decisions.

Reference that I'm looking at: screenshot from September here.

@mcjustin thanks, I will make the text change to "Observations". I think I found some issues with my changes when I tested some more and am fixing them now. I will ask Paul/Ivan to point dev instance to my branch when I am ready. Are you using https://confidentialbackend.embedhw.dev.cirg.uw.edu for testing EPIC stuff?

@achen2401 achen2401 marked this pull request as draft October 19, 2023 22:56
@achen2401
Copy link
Author

@mcjustin the changes have been deployed to https://confidentialbackend.embedhw.dev.cirg.uw.edu/ Let me know if you see any issue with my changes?

@achen2401 achen2401 removed the request for review from mcjustin June 5, 2024 16:02
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