-
Notifications
You must be signed in to change notification settings - Fork 347
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
Remove default selection of patient category #6354
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 #1318 ↗︎Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
@Ashesh3 fix the cypress |
LGTM |
@Ashesh3 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 a336f5f
The pull request makes the
category
andpatient_category
fields mandatory in the consultation and daily rounds forms, respectively, to ensure accurate and consistent data entry. It also fixes a bug in the default value ofpatient_category
in theDailyRounds
component.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
HOW
🤖 Generated by Copilot at a336f5f
category
andpatient_category
fields from "Comfort" to an empty string in theConsultationForm
andDailyRounds
components to avoid pre-selecting a category for the patient and to make it mandatory for the user to choose one (link, link, link, link, link)patient_category
field in theDailyRounds
component to check if it has a value and if not, add an error message and set theinvalidForm
flag to true (link)?.
) to check ifres.data
exists before accessing its properties in theDailyRounds
component to prevent errors ifres.data
is undefined or null (link)