Add Chlamydia support to test card #8389
Conversation
| }, | ||
| ]; | ||
|
|
||
| export default mockSupportedDiseaseTestPerformedChlamydia; |
There was a problem hiding this comment.
Might be worth a clean up PR to consolidate most of these mock files for the SupportedDiseaseTestPerformed entities
There was a problem hiding this comment.
Def worth a PR in my opinion. Maybe we can do it with the corresponding Chlamydia ticket since that should be the last STI?
| @@ -71,6 +71,8 @@ import { | |||
| baseStiAoeUpdateMock, | |||
| device10Id, | |||
| device10Name, | |||
| chlamydiaDeviceId, | |||
There was a problem hiding this comment.
IIRC we started using some of the device variables named with numbers like device7Name when we were still transitioning from the era of supporting only COVID/Flu/Multiplex. At the time of adding HIV support, we wanted to verify the ordering of the 6 default devices plus the new HIV device as the 7th device. We've continued this number naming for the other STIs to maintain consistency, but at this point it seems a little cumbersome to continue so for this I opted to use a more descriptive name. These names could also be something to address in a clean up PR if we'd like
|
mpbrown
requested review from
mehansen,
DanielSass,
emyl3,
DavidMcClatchey,
bobbywells52 and
arinkulshi-skylight
FRONTEND PULL REQUEST
Related Issue
Changes Proposed
Additional Information
mockSupportedDiseaseTestPerformed[diseaseNameHere]to mostly export from the same file instead of creating a new one for each new disease. Not terribly urgent thoughTesting
Big Organization,Test FacilityScreenshots / Demos