care icon indicating copy to clipboard operation
care copied to clipboard

Added more fields to Consultation Form

Open aeswibon opened this issue 2 years ago • 6 comments

Updates

  • [x] Fixes #859
  • [x] Related to coronasafe/care_fe#2218, coronasafe/care_fe#2738
  • [x] Added fields required in coronasafe/care_fe#3332
  • [x] Added fields required in coronasafe/care_fe#3333
  • [x] Added fields required in coronasafe/care_fe#3335
  • [x] Added fields required in coronasafe/care_fe#3336
  • [x] Added fields required in coronasafe/care_fe#3338
  • [x] Created a new Patient Health Details, model
  • [x] Added Allergies, Blood Group & Family History in Patient Health Details
  • [x] Added serializer, viewset, and API endpoint for Patient Health Details

aeswibon avatar Aug 15 '22 07:08 aeswibon

Waiting for FE before merging

gigincg avatar Sep 09 '22 14:09 gigincg

@cp-Coder to Add Screenshot of Swagger Docs for the new Consultation API

gigincg avatar Sep 09 '22 14:09 gigincg

image

aeswibon avatar Sep 13 '22 14:09 aeswibon

Let's not delete any fields immediately; let's keep a leeway of 1 month before deleting any data. remove any migration that deletes the columns.

vigneshhari avatar Sep 19 '22 17:09 vigneshhari

Codecov Report

Base: 56.08% // Head: 56.13% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (c164c4e) compared to base (e7052a9). Patch coverage: 51.97% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #979      +/-   ##
==========================================
+ Coverage   56.08%   56.13%   +0.05%     
==========================================
  Files         175      178       +3     
  Lines        8617     8689      +72     
  Branches     1460     1478      +18     
==========================================
+ Hits         4833     4878      +45     
- Misses       3732     3759      +27     
  Partials       52       52              
Impacted Files Coverage Δ
care/utils/notification_handler.py 16.80% <4.16%> (ø)
...e/facility/api/serializers/patient_consultation.py 32.50% <30.76%> (-1.60%) :arrow_down:
care/facility/api/viewsets/patient_otp.py 48.57% <33.33%> (-1.43%) :arrow_down:
...facility/api/serializers/patient_health_details.py 58.06% <58.06%> (ø)
care/facility/api/viewsets/patient.py 49.13% <61.90%> (+0.71%) :arrow_up:
care/facility/api/serializers/patient.py 56.93% <100.00%> (+0.91%) :arrow_up:
care/facility/models/json_schema/consultation.py 100.00% <100.00%> (ø)
care/facility/models/notification.py 100.00% <100.00%> (ø)
care/facility/models/patient.py 92.99% <100.00%> (+0.13%) :arrow_up:
care/facility/models/patient_base.py 100.00% <100.00%> (ø)
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov-commenter avatar Oct 03 '22 12:10 codecov-commenter

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
4.4% 4.4% Duplication

sonarqubecloud[bot] avatar Oct 19 '22 13:10 sonarqubecloud[bot]

Will open a new PR squashing the commits

aeswibon avatar Oct 22 '22 03:10 aeswibon