-
Notifications
You must be signed in to change notification settings - Fork 330
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
Sort "No Consultation Filed" patients to the top of Patient List page #1718
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1718 +/- ##
===========================================
- Coverage 62.20% 61.83% -0.37%
===========================================
Files 221 221
Lines 12204 12165 -39
Branches 1742 1736 -6
===========================================
- Hits 7591 7522 -69
- Misses 4305 4334 +29
- Partials 308 309 +1 ☔ View full report in Codecov by Sentry. |
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.
Would this approach work for consultations with readmissions?
@GokulramGHV it is not working properly for patient on readmission CC: @rithviknishad |
@GokulramGHV can you fix the issue ASAP |
Here is the conditions when we show the "No consultation filed" badge:
|
@Ashesh3 Thanks! |
@nihal467 Can you check if it works for the readmissions? I've updated the sorting order. |
LGTM |
Resolve conflicts |
Proposed Changes
Associated Issue
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