From 5292830224ea0cdc131753f02be97a70ed064445 Mon Sep 17 00:00:00 2001
From: smb2268 <sarah.breen@opentrons.com>
Date: Tue, 26 Mar 2024 10:10:49 -0400
Subject: [PATCH] Code review cleanup comments

---
 .../Devices/PipetteCard/FlexPipetteCard.tsx         | 13 ++++---------
 .../PipetteCard/__tests__/FlexPipetteCard.test.tsx  |  2 +-
 app/src/organisms/Devices/PipetteCard/index.tsx     |  2 +-
 3 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/app/src/organisms/Devices/PipetteCard/FlexPipetteCard.tsx b/app/src/organisms/Devices/PipetteCard/FlexPipetteCard.tsx
index 778f2f7d8d0..19a99233933 100644
--- a/app/src/organisms/Devices/PipetteCard/FlexPipetteCard.tsx
+++ b/app/src/organisms/Devices/PipetteCard/FlexPipetteCard.tsx
@@ -21,12 +21,7 @@ import { DropTipWizard } from '../../DropTipWizard'
 
 import { AboutPipetteSlideout } from './AboutPipetteSlideout'
 
-import type {
-  BadPipette,
-  Mount,
-  PipetteData,
-  HostConfig,
-} from '@opentrons/api-client'
+import type { BadPipette, Mount, PipetteData } from '@opentrons/api-client'
 import type { PipetteModelSpecs } from '@opentrons/shared-data'
 import type {
   PipetteWizardFlow,
@@ -66,7 +61,7 @@ export function FlexPipetteCard({
   isEstopNotDisengaged,
 }: FlexPipetteCardProps): JSX.Element {
   const { t, i18n } = useTranslation(['device_details', 'shared'])
-  const host = useHost() as HostConfig
+  const host = useHost()
 
   const [
     showAboutPipetteSlideout,
@@ -258,7 +253,7 @@ export function FlexPipetteCard({
           closeFlow={() => setShowDropTipWizard(false)}
         />
       ) : null}
-      {attachedPipette?.ok && showAboutPipetteSlideout && (
+      {attachedPipette?.ok && showAboutPipetteSlideout ? (
         <AboutPipetteSlideout
           pipetteId={attachedPipette.serialNumber}
           pipetteName={pipetteDisplayName ?? attachedPipette.instrumentName}
@@ -266,7 +261,7 @@ export function FlexPipetteCard({
           isExpanded={showAboutPipetteSlideout}
           onCloseClick={() => setShowAboutPipetteSlideout(false)}
         />
-      )}
+      ) : null}
       {showChoosePipette ? (
         <ChoosePipette
           proceed={handleAttach}
diff --git a/app/src/organisms/Devices/PipetteCard/__tests__/FlexPipetteCard.test.tsx b/app/src/organisms/Devices/PipetteCard/__tests__/FlexPipetteCard.test.tsx
index d9ce76ec2e5..3ea5096ea0d 100644
--- a/app/src/organisms/Devices/PipetteCard/__tests__/FlexPipetteCard.test.tsx
+++ b/app/src/organisms/Devices/PipetteCard/__tests__/FlexPipetteCard.test.tsx
@@ -4,6 +4,7 @@ import { describe, it, vi, beforeEach, afterEach, expect } from 'vitest'
 import { renderWithProviders } from '../../../../__testing-utils__'
 import { useCurrentSubsystemUpdateQuery } from '@opentrons/react-api-client'
 import { i18n } from '../../../../i18n'
+import { mockLeftSpecs } from '../../../../redux/pipettes/__fixtures__'
 import { handlePipetteWizardFlows } from '../../../PipetteWizardFlows'
 import { AboutPipetteSlideout } from '../AboutPipetteSlideout'
 import { FlexPipetteCard } from '../FlexPipetteCard'
@@ -11,7 +12,6 @@ import { ChoosePipette } from '../../../PipetteWizardFlows/ChoosePipette'
 import { DropTipWizard } from '../../../DropTipWizard'
 
 import type { PipetteData } from '@opentrons/api-client'
-import { mockLeftSpecs } from '../../../../redux/pipettes/__fixtures__'
 
 vi.mock('../../../PipetteWizardFlows')
 vi.mock('../../../PipetteWizardFlows/ChoosePipette')
diff --git a/app/src/organisms/Devices/PipetteCard/index.tsx b/app/src/organisms/Devices/PipetteCard/index.tsx
index de93a923734..0943f78311a 100644
--- a/app/src/organisms/Devices/PipetteCard/index.tsx
+++ b/app/src/organisms/Devices/PipetteCard/index.tsx
@@ -142,7 +142,7 @@ export const PipetteCard = (props: PipetteCardProps): JSX.Element => {
                   pipetteSpecs={pipetteModelSpecs}
                   mount={mount}
                   transform="scale(0.3)"
-                  transformOrigin={'20% 52%'}
+                  transformOrigin="20% 52%"
                 />
               </Flex>
             ) : null}