-
Notifications
You must be signed in to change notification settings - Fork 649
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
Mark patient as expired only after discharge in shifting #5404
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
LGTM |
@khavinshankar We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌. |
Kudos, SonarCloud Quality Gate passed!
|
WHAT
🤖 Generated by Copilot at da7b2d6
Refactor the shifting details update component to improve the discharge confirmation logic for expired patients. Fix the type definition of the
handleChange
function inShiftDetailsUpdate.tsx
.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at da7b2d6
FieldChangeEvent
type to use inhandleChange
function (link)handleSubmit
function to acceptdischarged
parameter and showDischargeModal
if patient is not discharged and status is "PATIENT EXPIRED" (link, link)afterSubmit
callback ofDischargeModal
to callhandleSubmit
withdischarged
set to true (link)onClick
handler ofSubmit
to callhandleSubmit
without arguments (link)