care_fe icon indicating copy to clipboard operation
care_fe copied to clipboard

Add IP Days Chip in Patient List and Patient Dashboard

Open konavivekramakrishna opened this issue 1 year ago • 14 comments

Proposed Changes

  • Fixes #6684
  • Added IP days chip in patient list and patient dashboard image image

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

konavivekramakrishna avatar Feb 22 '24 18:02 konavivekramakrishna

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

vercel[bot] avatar Feb 22 '24 18:02 vercel[bot]

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...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Feb 22 '24 18:02 netlify[bot]

@konavivekramakrishna image

  • [ ] Make the IP Days a circle badge on the consultation page similar to the MEWS score.

nihal467 avatar Feb 28 '24 06:02 nihal467

image

edge case when the year is 0202 it returns NaN

should this be handled?

konavivekramakrishna avatar Mar 01 '24 08:03 konavivekramakrishna

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.

github-actions[bot] avatar Mar 13 '24 04:03 github-actions[bot]

edge case when the year is 0202 it returns NaN

can be ignored. newer data wouldn't have weird years right.

rithviknishad avatar Mar 13 '24 04:03 rithviknishad

@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.

vercel[bot] avatar Mar 15 '24 02:03 vercel[bot]

: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.

github-actions[bot] avatar Mar 19 '24 11:03 github-actions[bot]

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.

github-actions[bot] avatar Apr 07 '24 04:04 github-actions[bot]

@konavivekramakrishna resolve the merge conflict

nihal467 avatar Apr 16 '24 13:04 nihal467

  • image
  • image

konavivekramakrishna avatar Apr 18 '24 13:04 konavivekramakrishna

@konavivekramakrishna image

the badge is missing text in the patient list view

nihal467 avatar Apr 23 '24 20:04 nihal467

I did't understand,

can you please specify what text it is missing,

IG everything is working as intended

konavivekramakrishna avatar Apr 24 '24 10:04 konavivekramakrishna

@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 avatar Apr 24 '24 12:04 rithviknishad

@rithviknishad can you push the two minor fix :

image

  • [ ] 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)

nihal467 avatar Apr 29 '24 09:04 nihal467

Closing as completed in #7735

rithviknishad avatar Apr 29 '24 10:04 rithviknishad