Incidents: Add optional patient impacted field

🚀 Feature Proposal

When recording a new incident or editing an existing one, add the ability to specify the impacted patient. This field will be optional.

Motivation

Some incidents impact patients while others will not. The system should provide a way to specify an impacted patient (if any)

Technical Details

Incident Model

Add an optional patient field:

patient?: string

Report Incident Form

Add an optional “Impacted Patient” field which updates the new patient field on the incident model The user must either specify an existing patient or leave the field blank

NOTE; See the Request Lab form for an example of how to add the patient form element to the report incident form

[rsousaj]

I don’t think I’m the best here to help you but I’d like to try.

From what I’ve used, the component receives onChange and onSearch as callbacks.

1. onSearch you pass a function that receive a string as argument and you return a Promise of the Search of that string from the Database.

2. onChange is when the user select a Patient. In this case, this callback receive a Patient as argument (actually an array of Patient but with just one Patient) and then you update the incidente state.

So, I didn’t understand why you need to display an error message. If the user type a search string that does not return a Patient, the field will be empty and perhaps its fine.

I hope I’ve helped.

Ps.: The Typeahead component is based on AsyncTypeahead from ‘react-bootstrap-typeahead’, if you want to know more.

[CreaTorAlexander]

I would like to take it