-
Notifications
You must be signed in to change notification settings - Fork 372
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 Patient Category in Daily Rounds and Consultation #3565
Adds Patient Category in Daily Rounds and Consultation #3565
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Requires backend PR ohcnetwork/care#1007 |
@rithviknishad merge the latest development branch and is it waiting for the backend to merge, if yes, remove need testing and label it as waiting for backend |
A minor bug was fixed, ready for testing. @nihal467 |
Kudos, SonarCloud Quality Gate passed!
|
@rithviknishad can you check the cypress test and other than that, PR looks good |
@rithviknishad please do check out the failing cypress test |
@khavinshankar @nihal467 all tests are passing when I ran cypress locally (with Staging BE). Workflow cypress is failing in many due to 403s. |
Functional changes as per #2210
Colors and styling updates will be made once #3458 is merged.
Proposed Changes
Adds mandatory patient category field in Patient Consultation form.
![image](https://user-images.githubusercontent.com/25143503/189366217-a9f2053a-3fd7-45a2-b458-dda4edbbda23.png)
Adds mandatory patient category field in Daily Rounds form.
![image](https://user-images.githubusercontent.com/25143503/189366088-ca053eb9-8406-400f-b5fa-d96c3e26cfb3.png)
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist