-
Notifications
You must be signed in to change notification settings - Fork 268
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
Adds M2M Consultation Diagnosis model #1690
Conversation
Functionality looks good to me @rithviknishad fix the test |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1690 +/- ##
==========================================
+ Coverage 60.05% 60.43% +0.38%
==========================================
Files 208 210 +2
Lines 11325 11473 +148
Branches 1607 1630 +23
==========================================
+ Hits 6801 6934 +133
- Misses 4300 4307 +7
- Partials 224 232 +8
☔ View full report in Codecov by Sentry. |
@sainak review required |
...cility/migrations/0393_rename_diagnosis_patientconsultation_deprecated_diagnosis_and_more.py
Show resolved
Hide resolved
Additional changes:
It never failed so far in other places because:
So the mixin has been renamed to: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LTGM
cc: @vigneshhari
LGTM |
Co-authored-by: Aakash Singh <mail@singhaakash.dev>
Proposed Changes
is_principle
verification_status
choice field (as per: https://www.hl7.org/fhir/valueset-condition-ver-status.html)ConsultationDiagnosis
table. icd11_diagnosis is mapped as Confirmed diagnosis and icd11_provisional_diagnosis is mapped as Provisional diagnosisPatientRelatedPermissionMixin
toConsultationRelatedPermissionMixin
(see comment for details)Associated Issue
Custom Migration
Architecture changes
Merge Checklist
/docs
Only PR's with test cases included and passing lint and test pipelines will be reviewed
@coronasafe/care-backend-maintainers @coronasafe/care-backend-admins