Skip to content
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

Show icu admission date on consultations dashboard #6779

Merged
merged 8 commits into from
Jan 4, 2024

Conversation

Pranshu1902
Copy link
Contributor

@Pranshu1902 Pranshu1902 commented Dec 2, 2023

WHAT

🤖[deprecated] Generated by Copilot at 8527c7b

Add admission date and ICU date to ConsultationUpdatesTab. This change enhances the patient's hospitalization information in the consultation details page.

Proposed Changes

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

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

HOW

🤖[deprecated] Generated by Copilot at 8527c7b

  • Add a new section to show admission date and ICU admission date and time of the patient (link)

@Pranshu1902 Pranshu1902 requested a review from a team as a code owner December 2, 2023 12:13
Copy link

vercel bot commented Dec 2, 2023

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 Dec 28, 2023 9:56am

Copy link

netlify bot commented Dec 2, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 202802c
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/658d4613775c6a0008adbe37
😎 Deploy Preview https://deploy-preview-6779--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.

@rithviknishad rithviknishad added hold waiting for related PR a co-related detail PR is under construction and removed hold labels Dec 5, 2023
@rithviknishad
Copy link
Member

The PR: #6719 has 2 changes that are conflicting to this PR.

  1. admission_date is renamed to encounter_date
  2. Label "Admission Date" has been white-labelled to encounter specific labels

@Pranshu1902 Pranshu1902 removed the waiting for related PR a co-related detail PR is under construction label Dec 27, 2023
@nihal467
Copy link
Member

nihal467 commented Dec 27, 2023

@Pranshu1902 as per the latest discussion on yesterdays EOD call, make the following changes

  • don't show any date or time ( admission, icu, discharge ) in the info card
  • show the admission and icu, time and date as same as the banner shown below when the patient is live
    image

note: currently the banner in the screenshot is shown only when the patient is discharge, so we are trying to implement the same when the patient is live

@Pranshu1902
Copy link
Contributor Author

@nihal467 updated view:
Live patient:
image

Discharged patient:
image

@nihal467
Copy link
Member

image

  • when the ICU admission details comes in, the whole UI looks weird
  • keep the whole thing in one row
  • keep the ICU admission on as same font as the admission on

@Pranshu1902
Copy link
Contributor Author

@nihal467 Updated view:
image

@nihal467
Copy link
Member

nihal467 commented Jan 1, 2024

LGTM

@khavinshankar khavinshankar merged commit 1a0bd2c into ohcnetwork:develop Jan 4, 2024
35 of 36 checks passed
Copy link

github-actions bot commented Jan 4, 2024

@Pranshu1902 We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show ICU Admission Date and Time in Discharge Summary / Consultation Page
4 participants