fix(2494): Incidents: Add the ability to edit incident details

[riiniii]

fix #2494

Fixes #2494.

Changes proposed in this pull request:

• merge code from master to current branch
• change useResolveIncident hook to useUpdateIncident
• in ViewIncidentDetails page, implement controlled inputs/forms with useState and useEffect, and make relevant fields editable:
  1a. department
  1b. category
  1c. category item
  1d. description
  1e. incident reported to
  1f. patient impacted
• make ViewIncidentDetails.test.tsx reflect that form should be editable
• make it so that patient field is always visible, with on view/update, allows user to update the patient field from previously null/empty/undeff to a new patient value
• add reportedTo

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/JDF9CwtuK2XJ28HSvFQGQmRx1fSe
✅ Preview: https://hospitalrun-frontend-git-fork-riiniii-fix-2494-3-hospitalrun.vercel.app

[riiniii]

cla was signed but showing as not signed, waiting on this

[riiniii]

@tehkapa could you add james + cynthia again for the reviewers? I can't seem to add them by myself

[DrewGregory]

@riiniii The issue with the CLA is that every author every commit in this PR has to sign the CLA. I would edit the commits to remove the authorship of Renee Huang, as it seems that they do not have a GitHub account and thus cannot sign the CLA. @tehkapa It seems like a good amount of contributors are running into this -- should the CLA only consider the GitHub author of the PR?

[DrewGregory]

I like that you made editing and resolving under the same mutation abstraction.

[DrewGregory]

Looks good. I would add a return type to getResolvedOn() so that it is clear to a user of this function what an "unresolved" result would look like.