Skip to content

Add filtering by patient status #2516

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
Merged
merged 3 commits into from
Nov 27, 2024
Merged

Add filtering by patient status #2516

merged 3 commits into from
Nov 27, 2024

Conversation

tvararu
Copy link
Member

@tvararu tvararu commented Nov 19, 2024 •
edited
Loading

This allows nurses to filter patients more easily.

TODO:

  • Fix duplicate statuses
  • Use smaller checkboxes

Descoped / later:

  • Display correct total number of patients in table when filtered

Screenshots

Screenshot 2024-11-27 at 16 59 36

@thomasleese thomasleese force-pushed the release-1c branch 4 times, most recently from 1b889f7 to 88a6d32 Compare November 22, 2024 14:02
@tvararu
Add available_states to PatientSessionStateConcern
f243df4 
This allows us to use the list of available states in places like the
patient filtering.
@tvararu
Add filtering by patient status
d6eb4b6 
This allows nurses to filter patients more easily.
@tvararu
Shrink checkboxes in patient filters
3ba2ab0 
Aligns the design to the prototype.
@tvararu tvararu temporarily deployed to mavis-pr-2516 November 27, 2024 16:59 Inactive
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@tvararu tvararu marked this pull request as ready for review November 27, 2024 16:59
@tvararu tvararu merged commit d9b1681 into release-1c Nov 27, 2024
9 checks passed
@tvararu tvararu deleted the status-filtering branch November 27, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thomasleese thomasleese thomasleese approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tvararu @thomasleese