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

Add MEWS Component #6760

Merged
merged 21 commits into from
Dec 13, 2023
Merged

Add MEWS Component #6760

merged 21 commits into from
Dec 13, 2023

Conversation

AshrafMd-1
Copy link
Contributor

@AshrafMd-1 AshrafMd-1 commented Nov 30, 2023

WHAT

🤖[deprecated] Generated by Copilot at 011d9f1

This pull request adds a new feature to display the Modified Early Warning Score (MEWS) of a patient on a ventilator in the PatientInfoCard.tsx component. MEWS is a tool that helps to identify patients who are at risk of deterioration and need urgent attention. The pull request introduces a new component Mews.tsx that calculates and displays the MEWS score based on the latest daily round data, which includes a new property consciousness_level. The pull request also updates the DailyRoundsModel interface to include the consciousness_level property.

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 011d9f1

  • Add a new component Mews that calculates and displays the Modified Early Warning Score (MEWS) for a patient based on their latest daily round data (link)
  • Import and render the Mews component in the PatientInfoCard component, passing the consultation ID as a prop (link, link)
  • Add the consciousness_level property to the DailyRoundsModel interface, which defines the shape of the daily round data (link)

@AshrafMd-1 AshrafMd-1 requested a review from a team as a code owner November 30, 2023 15:57
Copy link

vercel bot commented Nov 30, 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 12, 2023 11:28am

Copy link

netlify bot commented Nov 30, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 350f9d9
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/657843bfdea6fc0008cb3919
😎 Deploy Preview https://deploy-preview-6760--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.

Copy link
Member

@rithviknishad rithviknishad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@nihal467
Copy link
Member

nihal467 commented Dec 9, 2023

@AshrafMd-1 what is the update on the fix for the bug pointed out?

@AshrafMd-1
Copy link
Contributor Author

AshrafMd-1 commented Dec 9, 2023

@AshrafMd-1 what is the update on the fix for the bug pointed out?

@nihal467 , So in the previous EOD call, there were some changes

  • Make the mews component visible to all patients which have a log update
  • Use the recent or the lastest log update for mews
  • Add time of the log update

So I was just finishing the last task
I will push the pr in a few hours

@nihal467
Copy link
Member

@AshrafMd-1
image

  • the tooltip is small, increase the font size
  • the temperature is not recorded for the above patient, it is not showing in the tooltip, its missing

@nihal467 nihal467 added P1 breaking issue or vital feature and removed P2 labels Dec 12, 2023
Copy link
Member

@rithviknishad rithviknishad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AshrafMd-1 javascript is weird

Missing "temperature" is not being shown because of this

image

@nihal467
Copy link
Member

@AshrafMd-1 fix the issue ASAP, we need to get it merged by tomorrow EOD

@nihal467
Copy link
Member

LGTM

@khavinshankar khavinshankar merged commit 0adb29d into ohcnetwork:develop Dec 13, 2023
27 of 36 checks passed
@AshrafMd-1 AshrafMd-1 deleted the Fix-#6433 branch December 13, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 breaking issue or vital feature tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MEWS
4 participants