Add in patient id as a required field for the bulk uploader

[fzhao99]

Description

Our bulk upload sends don't include patient ID because on the COVID pipeline it used to be inferred in the RS transform if it wasn't included. We realized this was bad because making up an ID doesn't help with helping track down a patient if it's needed.

RS is making this a required field, so they're asking us to do this too. We'll need to add this as a required field in our uploader and to update the guidance flag to make it required.

Proposed solution

• Add in validation for a patient ID column (don't need to require the ID to take any shape, just as long as the sender has something in their system of record that works for them. See guidance / logic for accession number that we can steal from)
  • Should just require an extra field in the requiredFields list for the TestResultRow file validator.
• Update the guidance to also require this for bulk uploads. We'll just need to flip the flag from Requested --> Required and delete the qualifier around Some jurisdictions may require this field, ReportStream will notify you if this is the case.
• Mention to @jayna-SkylightDigital once this goes in so that we can let support know

[mindyatwork]

Product to check with RS on whether the Patient Field is required.

[mindyatwork]

Given the pivot away from storing patient data, we are closing this as wontfix.