-
Notifications
You must be signed in to change notification settings - Fork 3
BN-71 | Add. DiagnosesForm To Consultation Pad #23
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
Conversation
* BN-71 | Add. Utility functions to create FHIR entry and bundle * BN-71 | Refactor. Use utility functions to create consultation bundle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The item selection using keyboard is not working. When I do a down arrow on the filtered items and then hit Enter, always the first item is selected.
| const [isSubmitting, setIsSubmitting] = React.useState(false); | ||
|
|
||
| // DiagnosesForm state management | ||
| const [searchDiagnosesTerm, setSearchDiagnosesTerm] = React.useState(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The searchTerm, diagnosesErrors doesn't need to be a concern / part of consultation pad. This can be controlled within DiagnosisForm. Only the selectedDiagnosis can be part of ConsultationPad, which will be consumed during Consultation Bundle creation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have addressed this. Thanks for taking a look
| const { addNotification } = useNotification(); | ||
|
|
||
| // Use concept search hook for diagnoses | ||
| const { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, this can be moved into DiagnosisForm. The parent (ConsultationPad) component is only interested in selectedDiagnosis, whatever happens before that can be the components action itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have addressed this. Thanks for taking a look
| it('should render DiagnosesForm component after loading', async () => { | ||
| // Import concept service | ||
| // eslint-disable-next-line | ||
| const conceptService = require('@services/conceptService'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method of mocking seems to be inconsistent with how other services are mocked. Can we mock at beforeEach level ?
JIRA → BN-71
This PR introduces the ability to search, select, and manage diagnoses from the consultation pad as part of the patient’s clinical documentation. It lays the groundwork for diagnosis capture in the consultation workflow, in alignment with the FHIR
Conditionresource model. This implementation is also includes the diagnosis saving functionality.Key Features
Implementation Details
Diagnosis Search:
/ws/rest/v1/bahmni/terminologies/concepts.Certainty Selection:
ProvisionalandConfirmed.Diagnosis Management:
Internationalisation:
Testing & Type Definitions
Screenshots