-
Notifications
You must be signed in to change notification settings - Fork 549
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
Adds support for viewing discharged patients of a facility #7148
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Passing run #2187 ↗︎Details:
Review all test suite changes for PR #7148 ↗︎ |
is this expected behaviour @rithviknishad |
2764f0e
to
76071ff
Compare
@nihal467 unable to reproduce. I've admitted a previously discharged patient of "Critical Care" patient named "James Gunn" to "James Assets" facility (so he is a live patient). I'm still able to access the patient from the discharged patients view of "Critical Care" ![]() ![]() |
As per discussions, the following changes will be made to the permissions: Patient Registration PermissionCurrent: New: Consultation HistoryCurrent: New:
@aparnacoronasafe to comment on which solution to go with. |
Discussed this with @rithviknishad over slack. @nihal467 while testing, test for all these
|
LGTM |
@rithviknishad 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! 🙌 |
Required Backends
Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist