change the NIBP display from last 30 mins to current bed assignment date #6479

sriharsh05 · 2023-10-20T17:19:07Z

WHAT

`🤖 Generated by Copilot at 8c5f324`

This pull request adds the feature to show the blood pressure data from the HL7 vitals monitor for a consultation, based on the bed assignment history. It modifies the HL7PatientVitalsMonitor component, the api and models files, and the ConsultationUpdatesTab component. It also updates the ChannelOptions interface to include the consultationId property.
Changed the NIPB display in vitals monitor. It was showing only if the NIBP was measured 30 mins from current time. Now Changed it to display the measured NIBP from the current bed assignment date.

Proposed Changes

Fixes Vitals Monitor - NIBP: Only show if data is later than the current bed's assigned start date #6429
Changed the display timing of NIBP in vitals monitor.
used useQuery to implement the API request required to get the current bed of the patient.

@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

`🤖 Generated by Copilot at 8c5f324`

Add consultationId prop to HL7PatientVitalsMonitor component and use it to fetch and display the bed assignment history for the consultation (link, link, link, link, link, link, link, link)
Define consultation bed request and response interfaces in models.tsx file (link)

vercel · 2023-10-20T17:19:12Z

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	Nov 7, 2023 7:42am

netlify · 2023-10-20T17:19:14Z

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	`0535375`
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/6549ea2f12e50a00071ab6f5
😎 Deploy Preview	https://deploy-preview-6479--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.

rithviknishad

Changes in middleware is also required right?

src/Components/Facility/models.tsx

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx

…e model

src/Components/Facility/models.tsx

src/Components/VitalsMonitor/types.ts

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx

sriharsh05 · 2023-10-25T13:46:59Z

hey @rithviknishad
I have implemented the changes you requested. Can you review it please?

src/Redux/api.tsx

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx

src/Components/Facility/models.tsx

src/Components/Facility/ConsultationDetails/ConsultationUpdatesTab.tsx

rithviknishad

This behaviour should be present for central nursing station too

github-actions · 2023-10-31T11:17:09Z

👋 Hi, @sriharsh05,
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.

sriharsh05 · 2023-11-01T10:06:05Z

@nihal467
I have made the required changes. Now the patients NIBP data is reflecting in the patient consultation page.

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx

sriharsh05 · 2023-11-06T14:22:45Z

@rithviknishad
I have made the changes you requested.

rithviknishad

Nice! Very clean!

rithviknishad · 2023-11-06T17:20:42Z

@sriharsh05 the issue also mentions changes to be made to the middleware too. Are you working on it?

nihal467 · 2023-11-07T07:51:06Z

LGTM

sriharsh05 · 2023-11-07T08:47:14Z

@sriharsh05 the issue also mentions changes to be made to the middleware too. Are you working on it?

@rithviknishad I have not yet started working on it.

change the NIBP display from last 30 mins to currect bed assignment date

8c5f324

sriharsh05 requested a review from a team October 20, 2023 17:19

sriharsh05 requested a review from a team as a code owner October 20, 2023 17:19

sriharsh05 requested a review from mathew-alex October 20, 2023 17:19

vercel bot temporarily deployed to Preview October 20, 2023 17:19 Inactive

rithviknishad requested changes Oct 22, 2023

View reviewed changes

rithviknishad added changes required merge conflict pull requests with merge conflict labels Oct 22, 2023

sriharsh05 added 2 commits October 22, 2023 12:23

change model name to PascalCase and used PaginatedResponse to wrap th…

5315c82

…e model

use onResponse for useQuery

acbe0c3

vercel bot temporarily deployed to Preview October 22, 2023 07:26 Inactive

Merge branch 'develop' into fix#6429

12c504e

vercel bot temporarily deployed to Preview October 22, 2023 07:30 Inactive

resolved merge conflict

25e04a4

vercel bot temporarily deployed to Preview October 22, 2023 07:33 Inactive

move bed assignment date fetch logic to parent component

2d2a6f4

vercel bot temporarily deployed to Preview October 22, 2023 09:57 Inactive

replaced console log with notification for error

6378a4a

vercel bot temporarily deployed to Preview October 22, 2023 13:01 Inactive

rithviknishad removed the merge conflict pull requests with merge conflict label Oct 23, 2023

rithviknishad requested changes Oct 23, 2023

View reviewed changes

src/Components/Facility/models.tsx Outdated Show resolved Hide resolved

src/Components/VitalsMonitor/types.ts Outdated Show resolved Hide resolved

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx Outdated Show resolved Hide resolved

change logic by passing date to VitalsMonitor and change models

75addd7

vercel bot temporarily deployed to Preview October 23, 2023 07:44 Inactive

rithviknishad requested changes Oct 25, 2023

View reviewed changes

remove unused models and useState variables

e52c1ee

vercel bot temporarily deployed to Preview October 25, 2023 16:48 Inactive

rithviknishad requested changes Oct 26, 2023

View reviewed changes

github-actions bot added the merge conflict pull requests with merge conflict label Oct 31, 2023

sriharsh05 added 2 commits November 1, 2023 15:17

fix NIBP visibility bug in Vitals Monitor

365bf7a

merge develop to fix#6429

c6904e3

vercel bot temporarily deployed to Preview November 1, 2023 09:53 Inactive

github-actions bot removed the merge conflict pull requests with merge conflict label Nov 3, 2023

Merge branch 'develop' into fix#6429

66731de

vercel bot temporarily deployed to Preview November 6, 2023 11:31 Inactive

rithviknishad requested changes Nov 6, 2023

View reviewed changes

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx Outdated Show resolved Hide resolved

src/Components/VitalsMonitor/HL7PatientVitalsMonitor.tsx Outdated Show resolved Hide resolved

rithviknishad added changes required and removed test failed labels Nov 6, 2023

sriharsh05 and others added 2 commits November 6, 2023 19:35

delete unused function

4a34fb3

Merge branch 'develop' into fix#6429

8896de6

vercel bot temporarily deployed to Preview November 6, 2023 14:07 Inactive

rithviknishad approved these changes Nov 6, 2023

View reviewed changes

rithviknishad added needs testing and removed changes required labels Nov 6, 2023

Merge branch 'develop' into fix#6429

0535375

vercel bot temporarily deployed to Preview November 7, 2023 07:42 Inactive

nihal467 added tested and removed needs testing labels Nov 7, 2023

rithviknishad mentioned this pull request Nov 7, 2023

fixes #112; remove 30m timegate for BP ohcnetwork/teleicu_middleware#113

Merged

khavinshankar approved these changes Nov 7, 2023

View reviewed changes

khavinshankar merged commit e8091bf into ohcnetwork:develop Nov 7, 2023
33 of 35 checks passed

konavivekramakrishna mentioned this pull request Feb 2, 2024

fix review count ohcnetwork/leaderboard#198

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

change the NIBP display from last 30 mins to current bed assignment date #6479

change the NIBP display from last 30 mins to current bed assignment date #6479

sriharsh05 commented Oct 20, 2023 •

edited

Loading

vercel bot commented Oct 20, 2023 •

edited

Loading

netlify bot commented Oct 20, 2023 •

edited

Loading

rithviknishad left a comment •

edited

Loading

sriharsh05 commented Oct 25, 2023

rithviknishad left a comment

github-actions bot commented Oct 31, 2023

sriharsh05 commented Nov 1, 2023 •

edited

Loading

sriharsh05 commented Nov 6, 2023

rithviknishad left a comment

rithviknishad commented Nov 6, 2023

nihal467 commented Nov 7, 2023

sriharsh05 commented Nov 7, 2023

change the NIBP display from last 30 mins to current bed assignment date #6479

change the NIBP display from last 30 mins to current bed assignment date #6479

Conversation

sriharsh05 commented Oct 20, 2023 • edited Loading

WHAT

🤖 Generated by Copilot at 8c5f324

Proposed Changes

Merge Checklist

HOW

🤖 Generated by Copilot at 8c5f324

vercel bot commented Oct 20, 2023 • edited Loading

netlify bot commented Oct 20, 2023 • edited Loading

✅ Deploy Preview for care-egov-staging ready!

rithviknishad left a comment • edited Loading

Choose a reason for hiding this comment

sriharsh05 commented Oct 25, 2023

rithviknishad left a comment

Choose a reason for hiding this comment

github-actions bot commented Oct 31, 2023

sriharsh05 commented Nov 1, 2023 • edited Loading

sriharsh05 commented Nov 6, 2023

rithviknishad left a comment

Choose a reason for hiding this comment

rithviknishad commented Nov 6, 2023

nihal467 commented Nov 7, 2023

sriharsh05 commented Nov 7, 2023

sriharsh05 commented Oct 20, 2023 •

edited

Loading

`🤖 Generated by Copilot at 8c5f324`

`🤖 Generated by Copilot at 8c5f324`

vercel bot commented Oct 20, 2023 •

edited

Loading

netlify bot commented Oct 20, 2023 •

edited

Loading

rithviknishad left a comment •

edited

Loading

sriharsh05 commented Nov 1, 2023 •

edited

Loading