Reply feature for Doctor notes

[UdaySagar-Git]

Associated Issue

Frontend PR: ohcnetwork/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]

@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 65.42%. Comparing base (15debe1) to head (7397756).
Report is 1 commits behind head on develop.

Files with missing lines	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    65.41%   65.42%   +0.01%     
===========================================
  Files          242      242              
  Lines        13948    13965      +17     
  Branches      1931     1935       +4     
===========================================
+ Hits          9124     9137      +13     
- Misses        4443     4445       +2     
- Partials       381      383       +2     

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

[vigneshhari]

@sainak re-review required

[bodhish]

@sainak why are we not merging this?

[sainak]

@bodhish this needs a final review from vignesh