-
Notifications
You must be signed in to change notification settings - Fork 479
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
Patient Consultation: Route to Facility 🏥, Admission Date & Time 🕚 #6458
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 ↗︎
|
2 failed tests on run #1597 ↗︎
Details:
cypress/e2e/patient_spec/patient_crud.cy.ts • 2 failed testsReview all test suite changes for PR #6458 ↗︎ |
@nihal467 nope, that's not an intentional behaviour as per discussions made. My bad, I had accidentally forgot to remove the range constraints. |
@nihal467 ready for testing |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@rithviknishad functionality looks good to me, but the cypress and backend test are failing, update it |
@nihal467 backend tests fixed, cypress will require the backend to be merged to pass. |
👋 Hi, @rithviknishad, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@rithviknishad 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 🙌. |
WHAT
🤖 Generated by Copilot at 8be51d1
This pull request updates the UI and logic for the consultation form and details page, as well as the treatment summary and patient info card components. It adds new fields and options for the route to facility, ICU admission date, referral and transfer history, and treating physician. It also improves the date and time formatting and removes unused constants.
Required Backend PRs
verified_by
totreating_physician
care#1678Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at 8be51d1
CONSULTATION_STATUS
constant from theconstants.tsx
file (link)