fix: display hook details only in Hooks store confirm modal

cowprotocol · Jan 7, 2025 · 11c6863 · 11c6863
1 parent 5098421
commit 11c6863
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/apps/cowswap-frontend/src/modules/swap/containers/ConfirmSwapModalSetup/index.tsx b/apps/cowswap-frontend/src/modules/swap/containers/ConfirmSwapModalSetup/index.tsx
@@ -14,6 +14,7 @@ import {
   TradeConfirmation,
   TradeConfirmModal,
   useIsEoaEthFlow,
+  useIsHooksTradeType,
   useOrderSubmittedContent,
   useReceiveAmountInfo,
   useShouldPayGas,
@@ -71,6 +72,7 @@ export function ConfirmSwapModalSetup(props: ConfirmSwapModalSetupProps) {
   const appData = useAppData()
   const [userDeadline] = useUserTransactionTTL()
   const [isPartialApprove] = usePartialApprove()
+  const isHooksTradeType = useIsHooksTradeType()
 
   const slippageAdjustedSellAmount = trade?.maximumAmountIn(allowedSlippage)
   const isExactIn = trade?.tradeType === TradeType.EXACT_INPUT
@@ -127,6 +129,7 @@ export function ConfirmSwapModalSetup(props: ConfirmSwapModalSetupProps) {
         appData={appData || undefined}
         slippageAdjustedSellAmount={slippageAdjustedSellAmount}
         isPartialApprove={isPartialApprove}
+        displayHookDetails={isHooksTradeType}
       >
         {(restContent) => (
           <>

diff --git a/apps/cowswap-frontend/src/modules/trade/pure/TradeConfirmation/index.tsx b/apps/cowswap-frontend/src/modules/trade/pure/TradeConfirmation/index.tsx
@@ -52,6 +52,7 @@ export interface TradeConfirmationProps {
   children?: (restContent: ReactElement) => ReactElement
   slippageAdjustedSellAmount?: CurrencyAmount<Currency>
   isPartialApprove?: boolean
+  displayHookDetails?: boolean
 }
 
 export function TradeConfirmation(props: TradeConfirmationProps) {
@@ -81,6 +82,7 @@ export function TradeConfirmation(props: TradeConfirmationProps) {
     appData,
     isPartialApprove,
     slippageAdjustedSellAmount,
+    displayHookDetails,
   } = frozenProps || props
 
   /**
@@ -131,7 +133,7 @@ export function TradeConfirmation(props: TradeConfirmationProps) {
     onConfirm()
   }
 
-  const hookDetailsElement = (
+  const hookDetailsElement = displayHookDetails ? (
     <>
       {appData && (
         <OrderHooksDetails
@@ -144,7 +146,7 @@ export function TradeConfirmation(props: TradeConfirmationProps) {
         </OrderHooksDetails>
       )}
     </>
-  )
+  ) : null
 
   return (
     <styledEl.WidgetWrapper onKeyDown={(e) => e.key === 'Escape' && onDismiss()}>