Diagnosis: Adds field Principal Diagnosis #6218
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
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 configuration. |
4 tasks
rithviknishad
changed the title
github-actions
bot
added
the
cypress failed
|
@rithviknishad modify the consultation creation cypress test in patient_crud to have principle diagnosis as well |
|
LGTM, tested through @rithviknishad tunnel |
|
@khavinshankar only merge it, post backend and cypress pass's |
github-actions
bot
added
cypress passed
and removed
cypress failed
khavinshankar
approved these changes
Sep 6, 2023
|
@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 🙌. |
ShivamJhaa
pushed a commit
to ShivamJhaa/care_fe
that referenced
this pull request
Sep 6, 2023
* adds field principle diagnosis * show in consultation details * fix typo * fix cypress * try adding wait * fix tests
khavinshankar
pushed a commit
that referenced
this pull request
Sep 7, 2023
…6106) * lints * Added test for resource and migrated it to POM * trying a fix * nits * backend change * Reverted useless changes * fix test * Fixed failing test * Added api verification * Diagnosis: Adds field Principal Diagnosis (#6218) * adds field principle diagnosis * show in consultation details * fix typo * fix cypress * try adding wait * fix tests * open pdf file preview in new tab (#6226) * Changed weekly to average weekly for working hours (#6228) * Added padding to count block on patients page * fixed date format in asset manage page * Merged configure facility and configure health facility in one page * removed commented code * changed weekly to average weekly for working hours * Resolved comments --------- Co-authored-by: Mohammed Nihal <57055998+nihal467@users.noreply.github.com> Co-authored-by: Rithvik Nishad <mail@rithviknishad.dev> Co-authored-by: Ashesh <3626859+Ashesh3@users.noreply.github.com> Co-authored-by: Kshitij Verma <101321276+kshitijv256@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHAT
🤖 Generated by Copilot at a5fc8d2
This pull request adds a feature to capture and display the principal diagnosis of a consultation, following the ICD-11 standard. It modifies the
ConsultationFormcomponent to include a new required field, theConsultationDetailscomponent to show the diagnosis, and theConsultationModelinterface to store the diagnosis id.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at a5fc8d2
ShowDiagnosisto display the principal diagnosis of the consultation (link)AutocompleteFormFieldcomponent to create a searchable dropdown field for selecting the principal diagnosis (link)icd11_principal_diagnosisto theFormDetailstype and theConsultationModelinterface to store the id of the selected diagnosis object (link, link)icd11_principal_diagnosisproperty to undefined in theinitFormfunction (link)icd11_principal_diagnosisfield in thevalidateFormfunction, which requires the field to be non-empty and match one of the final or provisional diagnoses (link)icd11_principal_diagnosisproperty to the payload object that is sent to the API when updating or creating a consultation (link)icd11_principal_diagnosisproperty to undefined when the final or provisional diagnoses arrays are changed in the form (link)AutocompleteFormFieldcomponent to the consultation form, with the nameicd11_principal_diagnosisand the labelPrincipal Diagnosis, using the final or provisional diagnoses arrays as the options (link)