care icon indicating copy to clipboard operation
care copied to clipboard

Prescriptions and MAR: Disallow users to perform edits on discharged encounters

Open rithviknishad opened this issue 9 months ago • 3 comments

Proposed Changes

  • Disallow changes to Prescription and Medicine Administration Records for discharged encounters.
  • Merge prescription related tests to a single file and have dedicated test file for medicine administrations

Associated Issue

  • [x] Fixes https://github.com/coronasafe/care_fe/issues/7755
  • [x] Fixes #2105

Merge Checklist

  • [x] Tests added/fixed
  • [x] Linting Complete

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

rithviknishad avatar May 07 '24 11:05 rithviknishad

Waiting for confirmation regd. whether all remote users needs to be restricted or just remote nurses.

rithviknishad avatar May 13 '24 05:05 rithviknishad

Confirmed. Doctors and Nurses of home facility only must be able to edit consultation related data. Other than doctor notes.

rithviknishad avatar May 13 '24 05:05 rithviknishad

@rithviknishad any update on this, is it good for review

nihal467 avatar Jun 04 '24 18:06 nihal467

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Project coverage is 65.41%. Comparing base (5bb466b) to head (d6f72ab). Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
care/facility/api/serializers/prescription.py 75.00% 1 Missing and 1 partial :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2133      +/-   ##
===========================================
+ Coverage    65.39%   65.41%   +0.01%     
===========================================
  Files          242      242              
  Lines        13935    13948      +13     
  Branches      1927     1931       +4     
===========================================
+ Hits          9113     9124      +11     
- Misses        4442     4443       +1     
- Partials       380      381       +1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jul 09 '24 10:07 codecov[bot]

@rithviknishad Waiting for your changes

gigincg avatar Aug 05 '24 01:08 gigincg