Reply feature for Doctor notes

[UdaySagar-Git]

Associated Issue

Frontend PR: coronasafe/care_fe#7805

Architecture changes

• Remove this section if not used

Merge Checklist

• Tests added/fixed
• Update docs in /docs
• Linting Complete
• Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed

@coronasafe/care-backend-maintainers @coronasafe/care-backend-admins

[nihal467]

@UdaySagar-Git fix the lint and test in the PR

[UdaySagar-Git]

@nihal467 fixed !

[sainak]

@UdaySagar-Git how will you ensure that the notes that you fetch that are replies include the parent in response?, also what if the notes have replies, but it were not fetched in the current response

cc @rithviknishad

[UdaySagar-Git]

@sainak , thanks for pointing that out. It won't fetch the parent if there are more notes. Instead of sending just the ID, sending the complete parent object along with the note would solve this. Should I consider that and make changes in frontend according to it?

[rithviknishad]

@UdaySagar-Git we could add a ReplyToPatientNoteSerializer containing a lesser number of fields, just the required ones.

[rithviknishad]

Other than this, LGTM.

Tests would be appreciated 😄

[rithviknishad]

Tests needed

[UdaySagar-Git]

@rithviknishad forgot to commit 😅 ,
just added

[nihal467]

LGTM

[nihal467]

@UdaySagar-Git can you make the changes requested

[nihal467]

@UdaySagar-Git what is the update on the PR

[UdaySagar-Git]

@nihal467 this pr is ready for a final review

[UdaySagar-Git]

@vigneshhari can you please review this , added a extra validation check for consultation
since the doctor notes will be specfic to consultation and is required

[UdaySagar-Git]

@nihal467 can you try re-running the tests now ?

[codecov]

Codecov Report

Attention: Patch coverage is 76.47059% with 4 lines in your changes missing coverage. Please review.

Project coverage is 63.50%. Comparing base (1912662) to head (386102f).

Files	Patch %	Lines
care/facility/api/serializers/patient.py	75.00%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2160      +/-   ##
===========================================
+ Coverage    63.48%   63.50%   +0.01%     
===========================================
  Files          238      238              
  Lines        13264    13281      +17     
  Branches      1886     1890       +4     
===========================================
+ Hits          8421     8434      +13     
- Misses        4490     4492       +2     
- Partials       353      355       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[vigneshhari]

@sainak re-review required