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.

Sign up for GitHub

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

Jump to bottom

Facility listing- occupancy #8055

Open
2 tasks
aparnacoronasafe opened this issue Jun 19, 2024 · 3 comments · May be fixed by #8097
Open
2 tasks

Facility listing- occupancy #8055

aparnacoronasafe opened this issue Jun 19, 2024 · 3 comments · May be fixed by #8097
Labels
Backend blocked Frontend good first issue Good for newcomers work-in-progress

Comments

@aparnacoronasafe
Copy link
Member

Is your feature request related to a problem? Please describe.
Today, the facility listing (https://care.coronasafe.in/facility) shows the total occupancy of the hospital. ICU occupancy needs to be shown separately

Describe the solution you'd like

  • In the facility listing (https://care.coronasafe.in/facility) add a second label with ICU occupancy. Keep the design consistent with "occupancy". Show "number of patients admitted to ICU bed type/ number of ICU beds registered"
  • Occupancy label today shows total live patients/total beds registered. Instead, correct it to "total patients marked as admitted in consultation/ total no. of registered beds"
@Ni-011
Copy link

Ni-011 commented Jun 23, 2024

Hey I made the changes to ICU occupancy and Occupancy, since the text is too long I wrapped it around is this ui fine?

image

image

@Fredrick2503
Copy link

Hey can you assign me this issue I would like to solve it
@aparnacoronasafe

@Fredrick2503 Fredrick2503 linked a pull request Jul 1, 2024 that will close this issue
Open
6 tasks
Fredrick2503 added a commit to Fredrick2503/care_fe that referenced this issue Jul 1, 2024
@Fredrick2503
Merge branch 'develop' into issues/coronasafe#8055/Facility_listing_o…
17f6fcb 
…ccupancy
Fredrick2503 added a commit to Fredrick2503/care_fe that referenced this issue Jul 1, 2024
@Fredrick2503
Merge branch 'develop' into issues/coronasafe#8055/Facility_listing_o…
111c169 
…ccupancy
Fredrick2503 added a commit to Fredrick2503/care_fe that referenced this issue Jul 2, 2024
@Fredrick2503
Merge branch 'issues/coronasafe#8055/Facility_listing_occupancy' of h…
9be9ba9 
…ttps://github.com/Fredrick2503/care_fe into issues/coronasafe#8055/Facility_listing_occupancy

pull from main repo
Fredrick2503 added a commit to Fredrick2503/care_fe that referenced this issue Jul 2, 2024
@Fredrick2503
Merge branch 'coronasafe:develop' into issues/coronasafe#8055/Facilit…
b01d259 
…y_listing_occupancy
Fredrick2503 added a commit to Fredrick2503/care_fe that referenced this issue Jul 2, 2024
@Fredrick2503
Merge branch 'issues/coronasafe#8055/Facility_listing_occupancy' of h…
16d26a4 
…ttps://github.com/Fredrick2503/care_fe into issues/coronasafe#8055/Facility_listing_occupancy

pull from repo
@gigincg gigincg added the blocked label Jul 9, 2024
@gigincg
Copy link
Member

gigincg commented Jul 9, 2024

@aparnacoronasafe This seems overkill, this would lead to way too much data on each card. Can you throw some light on what we're solving with this.

This data is available in the dashboards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Backend blocked Frontend good first issue Good for newcomers work-in-progress
Projects
Care
Status: Triage
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Added ICU occupancy badge Fredrick2503/care_fe
5 participants
@gigincg @khavinshankar @Ni-011 @aparnacoronasafe @Fredrick2503