-
Notifications
You must be signed in to change notification settings - Fork 423
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
Shows encounter duration in discharge confirmation dialog #8124
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 #2834 ↗︎
Details:
Review all test suite changes for PR #8124 ↗︎ |
@rithviknishad the IP no of patients are shown as 24 days and in the discharge pop-up it is shown as 25 days, how are we calculating the no of days, is it based on 24 hours clock from the date of admission or is it per night, we need to clarify it with @aparnacoronasafe and modify the PR based on the logic finalised . My suggestion its better to keep logic based on how rooms days are calculated in hospital as we are moving towards billing and schedule |
As discussed with @aparnacoronasafe , we've decided to show the n'th day (i.e. the logic in the discharge popup is correct and the IP Days number needs to be incremented by 1) i.e.: |
LGTM |
@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