Skip to content
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

General instructions (advice): consultation #8365

Closed
aparnacoronasafe opened this issue Aug 20, 2024 · 2 comments · Fixed by #8368
Closed

General instructions (advice): consultation #8365

aparnacoronasafe opened this issue Aug 20, 2024 · 2 comments · Fixed by #8368
Assignees
Labels
good first issue Good for newcomers

Comments

@aparnacoronasafe
Copy link
Member

Is your feature request related to a problem? Please describe.
In consultation form (https://care.coronasafe.in/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/patient/7e8fc20f-02da-47f7-8c33-f2d8c7e52dd8/consultation/edfe0bd6-0f50-4a4c-8cd0-2d4ad0e9cea1/update) the field for General Instructions (advise) was made mandatory field as that was the field where doctor would detail out the care plan. But now we have specific fields to detail out the care plan.

Therefore the text field may be made non-mandatory to easy data entry

Describe the solution you'd like
Make General Instructions (advise) field non-mandatory

@github-project-automation github-project-automation bot moved this to Triage in Care Aug 20, 2024
@aparnacoronasafe aparnacoronasafe added the good first issue Good for newcomers label Aug 20, 2024
@JOSHIK27
Copy link
Contributor

image

Made the field General Instructions non mandatory as asked. Simply removed the required condition on text element and deleted error message handling part for general instructions field.

This is a minor add on (deletion of few lines) to this pr #8360 @rithviknishad

@rithviknishad
Copy link
Member

Feel free to create a separate PR @JOSHIK27 even if it's just 1-2 lines of non-conflicting change 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants