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 Patient details of selected Preset in Location Live Feed Monitoring #6667

Open
rithviknishad opened this issue Nov 20, 2023 · 7 comments
Assignees
Labels

Comments

@rithviknishad
Copy link
Member

Describe the bug
Currently, we are not showing the patient in the selected bed in Live Feed monitoring as there's no efficient way to fetch which patient is present on a particular bed.

To Reproduce
Steps to reproduce the behavior:

  1. Go to a location live feed monitoring page

Expected behavior
Show Patient name, age gender as shown in the mockup

Screenshots
image

@github-project-automation github-project-automation bot moved this to Triage in Care Nov 20, 2023
@rithviknishad rithviknishad moved this from Triage to Wishlist in Care Nov 20, 2023
@kirtanchandak
Copy link

Hey, can I work on this issue?

@rithviknishad rithviknishad removed blocked waiting for related PR a co-related detail PR is under construction labels Feb 21, 2024
@rash-27
Copy link
Contributor

rash-27 commented Apr 3, 2024

If no one is working on this , I would like to work on this issue

@rithviknishad
Copy link
Member Author

@rash-27 could you describe how you'll be trying to solve this?

@rash-27
Copy link
Contributor

rash-27 commented Apr 3, 2024

@rithviknishad Currently, we are fetching information for assets from [api/v1/assets] to display the feed
Screenshot from 2024-04-03 16-18-12
. Instead, if we try to fetch from [/api/v1/facility/{facility_external_id}/patient_asset_beds/], we get the additional information of the patient name too, and if we try to include the patient's age in the route in the backend I think we can solve this, Is my approach right?

@rithviknishad
Copy link
Member Author

Yup, you can try that! You need not change the backend to provide the age. We can calculate age from FE itself using the recently added utility function right?

@rash-27
Copy link
Contributor

rash-27 commented Apr 3, 2024

Yup , I'll try it .

@noufalrahim
Copy link
Contributor

@rithviknishad can u assign this to me..?

@github-actions github-actions bot added needs-triage question Further information is requested labels Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Wishlist
Development

Successfully merging a pull request may close this issue.

4 participants