-
Notifications
You must be signed in to change notification settings - Fork 412
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 IP Days chip for admission encounters #7735
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Passing run #2449 ↗︎
Details:
Review all test suite changes for PR #7735 ↗︎ |
LGTM |
LGTM |
@@ -577,6 +577,16 @@ export const PatientManager = () => { | |||
text="Readmission" | |||
/> | |||
)} | |||
{patient.last_consultation?.suggestion === "A" && | |||
patient.last_consultation.facility === patient.facility && |
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.
what is this (patient.last_consultation.facility === patient.facility
) check for? seems redundant!
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.
It is to check whether the last_consultation is same as patient's active facility or not.
Since patient could be transferred to another facility without a consultation filed yet. So in UI it'd be shown as "No Consultation Filed" but shows IP days as per last consultation which is already closed consultation.
Ref: #7252 (comment)
@rithviknishad Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist