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

Patient generalData flow refractor and PatientInfo View #64

Merged
merged 14 commits into from
Mar 11, 2024

Conversation

ninaboord
Copy link
Contributor

@ninaboord ninaboord commented Mar 10, 2024

Patient generalData flow refractor and PatientInfo View

♻️ Current situation & Problem

Before this PR, we did not have a way for the patient to enter or change their personal information if they were not connected to FHIRStore.

  • If the patient is NOT connected to HealthKit, PatientInfo View allows patient to update their general data (name, dob, age..), and view updates global patient variables
  • If the patient IS connected to HealthKit, PatientInfo View will be auto-populated and will allow the Patient to edit their data if they wish
  • Global variables now passed into LLM, so LLMInteraction code is now cleaner.
  • The Chief Complaint is now AFTER the forms. This is to set up for the next bonus pull request of passing in patient medical history into the LLM.

⚙️ Release Notes

FHIRStore extraction functionality that I did in LLMInteraction is now moved to PatientInfo view. The functions calculateAge, getValue, and getInfo functions are now inside of a view, and loadData() is a private function that then updates the data onAppear of the PatientInfo view.

Screenshot 2024-03-10 at 8 47 00 PM

📚 Documentation

n/a

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Attention: Patch coverage is 28.26087% with 132 lines in your changes are missing coverage. Please review.

Project coverage is 10.96%. Comparing base (28194c1) to head (a55757d).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
- Coverage   12.09%   10.96%   -1.13%     
==========================================
  Files          59       60       +1     
  Lines        3261     3333      +72     
==========================================
- Hits          394      365      -29     
- Misses       2867     2968     +101     
Files Coverage Δ
Intake/Home.swift 92.16% <100.00%> (+0.08%) ⬆️
Intake/ChiefComplaint/SummaryView.swift 0.00% <0.00%> (ø)
Intake/SocialHistory/SmokingHistory.swift 0.00% <0.00%> (ø)
Intake/Surgery/SurgeryView.swift 0.00% <0.00%> (ø)
Intake/ScrollablePDF.swift 0.00% <0.00%> (ø)
Intake/Elements.swift 25.59% <64.71%> (-12.88%) ⬇️
Intake/ChiefComplaint/LLMInteraction.swift 0.00% <0.00%> (-35.84%) ⬇️
Intake/EditPatient.swift 0.00% <0.00%> (ø)
Intake/General Data View/PatientInfo.swift 30.71% <30.71%> (ø)

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 28194c1...a55757d. Read the comment docs.

Copy link
Contributor

@nriedman nriedman left a comment

Choose a reason for hiding this comment

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

Well done!

@ninaboord ninaboord changed the title Patient generalData flow refractor Patient generalData flow refractor and PatientInfo View Mar 11, 2024
@ninaboord ninaboord merged commit a8de6e9 into main Mar 11, 2024
7 checks passed
@ninaboord ninaboord deleted the personal-data branch March 11, 2024 03:44
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