Skip to content
This repository has been archived by the owner on May 2, 2022. It is now read-only.

#129 Connect Symptoms Update and Doctor/Patient #148

Merged
merged 34 commits into from
Mar 28, 2022

Conversation

MilitsaB
Copy link
Collaborator

Related Issue

Proposed Changes (Description)

  • Add Connection Between Symptoms and Doctor View
  • Dynamic Doctor table for patients
  • View on Patient Dashboard to see doctor status

What will be affected:

  • Patient Dashboard
  • Medical User Dashboard and Patient view

Checklist

  • Tests
  • Documentation
  • Build Successful
  • Follow Coding Guidelines

@github-actions
Copy link

github-actions bot commented Mar 27, 2022

Visit the preview URL for this PR (updated for commit 27b7c49):

https://soen-390-ba781--pr148-129-connect-symptom-omac1ssr.web.app

(expires Mon, 04 Apr 2022 18:27:31 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@martinsenecal martinsenecal self-requested a review March 27, 2022 22:09
@KrisTheCanadian
Copy link
Owner

KrisTheCanadian commented Mar 27, 2022

Codecov Report

Merging #148 (27b7c49) into dev (f61805b) will decrease coverage by 3.72%.
The diff coverage is 40.48%.

@@            Coverage Diff             @@
##              dev     #148      +/-   ##
==========================================
- Coverage   56.74%   53.03%   -3.71%     
==========================================
  Files          50       57       +7     
  Lines        1336     1522     +186     
  Branches      345      387      +42     
==========================================
+ Hits          758      807      +49     
- Misses        574      708     +134     
- Partials        4        7       +3     
Impacted Files Coverage Δ
...s/dashboard/AdminDashboard/AdminTable/EditUser.tsx 3.34% <0.00%> (ø)
...mponents/dashboard/AdminDashboard/PatientTable.tsx 0.00% <0.00%> (ø)
...ashboard/AdminDashboard/UnassignedPatientTable.tsx 0.00% <0.00%> (ø)
...c/components/dashboard/MedicalView/MedicalView.tsx 70.00% <ø> (-2.72%) ⬇️
...c/components/dashboard/PatientView/TestResults.tsx 0.00% <0.00%> (ø)
...s/dashboard/ThirdPartyView/ThirdPartyDashboard.tsx 100.00% <ø> (ø)
...UpdateSymptomsDesign/SymptomsIntensityQuestion.tsx 2.23% <ø> (ø)
src/pages/auth/admincreateaccount.tsx 48.34% <0.00%> (ø)
...c/components/dashboard/PatientView/PatientView.tsx 3.85% <3.85%> (ø)
...ts/dashboard/PatientView/PatientMedicalConnect.tsx 7.15% <7.15%> (ø)
... and 18 more

Continue to review full report at Codecov.

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

@MilitsaB MilitsaB force-pushed the #129-Connect-SymptomsUpdate-and-Doc/Patient branch from 62e163e to d31c9a1 Compare March 27, 2022 23:02
@KrisTheCanadian KrisTheCanadian self-requested a review March 28, 2022 00:09
Copy link
Owner

@KrisTheCanadian KrisTheCanadian left a comment

Choose a reason for hiding this comment

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

We should probably not comment out part of the tests

src/test/patientInfo.test.tsx Outdated Show resolved Hide resolved
@MilitsaB MilitsaB force-pushed the #129-Connect-SymptomsUpdate-and-Doc/Patient branch from ea4e1c7 to 5fadba4 Compare March 28, 2022 00:22
Alexialsousa
Alexialsousa previously approved these changes Mar 28, 2022
@martinsenecal martinsenecal force-pushed the #129-Connect-SymptomsUpdate-and-Doc/Patient branch from 1262472 to ef0f8c7 Compare March 28, 2022 02:43
gkillick
gkillick previously approved these changes Mar 28, 2022
martinsenecal
martinsenecal previously approved these changes Mar 28, 2022
src/components/App.tsx Outdated Show resolved Hide resolved
…Doc/Patient' into #129-Connect-SymptomsUpdate-and-Doc/Patient
@MilitsaB MilitsaB merged commit afd3b9e into dev Mar 28, 2022
@MilitsaB MilitsaB deleted the #129-Connect-SymptomsUpdate-and-Doc/Patient branch March 28, 2022 21:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants