care_fe
care_fe copied to clipboard
Add IP Days Chip in Patient List and Patient Dashboard
Proposed Changes
- Fixes #6684
- Added IP days chip in patient list and patient dashboard
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
care-storybook | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Mar 4, 2024 2:22pm |
Deploy Preview for care-egov-staging ready!
Name | Link |
---|---|
Latest commit | b651e30be4574d342dbf6c5f95315b57bbfd1913 |
Latest deploy log | https://app.netlify.com/sites/care-egov-staging/deploys/662698fbd4c11f00086b1ca5 |
Deploy Preview | https://deploy-preview-7252--care-egov-staging.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@konavivekramakrishna
- [ ] Make the IP Days a circle badge on the consultation page similar to the MEWS score.
edge case when the year is 0202 it returns NaN
should this be handled?
Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.
edge case when the year is 0202 it returns NaN
can be ignored. newer data wouldn't have weird years right.
@konavivekramakrishna is attempting to deploy a commit to the Open Healthcare Network Team on Vercel.
A member of the Team first needs to authorize it.
:wave: Hi, @konavivekramakrishna, Conflicts have been detected against the base branch. Please rebase your branch against the base branch.
This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.
Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions.
@konavivekramakrishna resolve the merge conflict
@konavivekramakrishna
the badge is missing text in the patient list view
I did't understand,
can you please specify what text it is missing,
IG everything is working as intended
@nihal467 could you elaborate a bit more? Everything seems fine to me, IP Days are present in the patient list as well as consultation dashboard.
@rithviknishad can you push the two minor fix :
- [ ] add a prefix in the patient list page "IP Days"
- [ ] only show this badge if the patient is having a live consultation
- [ ] once the patient is discharged, in the consultation page retain number of days badge (not working now)
Closing as completed in #7735