-
Notifications
You must be signed in to change notification settings - Fork 426
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
CNS: filter by locations changes #8812
CNS: filter by locations changes #8812
Conversation
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
So we do need to make an extra change, consider the following scenario:
Now, consider following behavior. (1) User selects "Include monitors without patients" At present, the filter form selection stays the same (resulting in No vitals screen, which is not accurate), so we need to reset the location selection whenever user marks/unmarks the checkbox. @rithviknishad I'm wondering if there's a way to do that without touching LocationSelect? |
Resolved 👍 |
LGTM |
👋 Hi, @Jacobjeevan, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@Jacobjeevan Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
@ohcnetwork/care-fe-code-reviewers
Merge Checklist