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

Correct the occupancy details in facility card #5533

Closed
Bhavik-ag opened this issue May 26, 2023 · 6 comments · Fixed by #5539
Closed

Correct the occupancy details in facility card #5533

Bhavik-ag opened this issue May 26, 2023 · 6 comments · Fixed by #5539

Comments

@Bhavik-ag
Copy link
Contributor

Describe the bug
The occupancy of the bed shown on the facility page for a facility card is currently shown equal to patient_count / bed_count.

To Reproduce
Steps to reproduce the behavior:

  1. Go to facility tab

Expected behavior
Occupancy should be equal to no of occupied_bed_count / bed_count.

Screenshots
2023-05-26_12-08

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@rithviknishad
Copy link
Member

Hey @Bhavik-ag this is not a bug and is expected behaviour. After discussions, it was decided to show live patients / total bed count. Not occupied beds / total beds.

cc: @nihal467 @aparnacoronasafe

@rithviknishad
Copy link
Member

We could add a tooltip maybe saying: Live Patients / Total beds?

@Bhavik-ag
Copy link
Contributor Author

Hey @rithviknishad Instead of one chip for occupancy, we can show two chips one for number of beds and one for live patients?

@rithviknishad
Copy link
Member

We tweaked and tried lots of designs during the calls. 😅
We could discuss this, can you share this in our Slack channel in care general so that we can get everyone's suggestions?

@nihal467
Copy link
Member

@Bhavik-ag Can you add a tooltip for that chip? It would be helpful if you could complete it by 5:00 p.m. this evening since tonight is our weekly production release.

@Bhavik-ag
Copy link
Contributor Author

Ya sure!! @nihal467 Working on this

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

Successfully merging a pull request may close this issue.

3 participants