fix(protocol-designer): fix newlocation display name in StepSummary (#…

…16727)

For moveLabware StepSummary, we need grab the correct newLocation
display name if the newLocation is not a slot. Here, I check for the
newLocation type and retrieve the appropriate display name.

Closes RQA-3420

protocol-designer/src/assets/localization/en/protocol_steps.json

@@ -69,6 +69,7 @@
   "multiAspirate": "Consolidate path",
   "multiDispense": "Distribute path",
   "new_location": "New location",
+  "off_deck": "Off-Deck",
   "pause": {
     "untilResume": "Pausing until manually told to resume",
     "untilTemperature": "<text>Pausing until</text><semiBoldText>{{module}}</semiBoldText><text>reaches</text><tag/>",

protocol-designer/src/pages/Designer/ProtocolSteps/StepSummary.tsx

@@ -272,6 +272,14 @@ export function StepSummary(props: StepSummaryProps): JSX.Element | null {
     case 'moveLabware':
       const { labware, newLocation, useGripper } = currentStep
       const labwareName = labwareNicknamesById[labware]
+      let newLocationName = newLocation
+      if (newLocation in modules) {
+        newLocationName = getModuleDisplayName(modules[newLocation].model)
+      } else if (newLocation in labwareEntities) {
+        newLocationName = labwareNicknamesById[newLocation]
+      } else if (newLocation === 'offDeck') {
+        newLocationName = t('off_deck')
+      }
       stepSummaryContent = (
         <StyledTrans
           i18nKey={
@@ -282,7 +290,7 @@ export function StepSummary(props: StepSummaryProps): JSX.Element | null {
           values={{
             labware: labwareName,
           }}
-          tagText={newLocation}
+          tagText={newLocationName}
         />
       )
       break