-
Notifications
You must be signed in to change notification settings - Fork 292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch investigation default from dict
to list
and remove deprecated prescriptions
#2410
Switch investigation default from dict
to list
and remove deprecated prescriptions
#2410
Conversation
…ted `prescriptions`
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2410 +/- ##
===========================================
- Coverage 64.91% 64.90% -0.01%
===========================================
Files 242 242
Lines 13811 13809 -2
Branches 1906 1906
===========================================
- Hits 8965 8963 -2
Misses 4468 4468
Partials 378 378 ☔ View full report in Codecov by Sentry. |
@sainak @vigneshhari docker compose exec backend bash -c "python manage.py populate_investigations" command is not working in docker app windows |
@nihal467 could you share more details on what the error is? Unable to replicate. |
LGTM |
the PR is working well, it's just the investigation loading command for docker is not working, but loaded them using some alternative way aakash suggested and the PR is good to be merged |
Proposed Changes
list
as default forPatientConsultation.investigation
instead ofdict
prescriptions
column from PatientConsultation model andmedication_given
(prescriptions) from Daily Rounds model which was missing from remove deprecated fields that has been migrated to new models (Diagnosis, Symptoms and Prescriptions) #2247Associated Issue
Merge Checklist
/docs
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins