fix(limit-orders): price updated warning should not be displayed (cow…

…protocol#3925)
xyzseer · Feb 26, 2024 · 7a5b64d · 7a5b64d
1 parent bf8998d
commit 7a5b64d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/apps/cowswap-frontend/src/modules/limitOrders/containers/LimitOrdersConfirmModal/index.tsx b/apps/cowswap-frontend/src/modules/limitOrders/containers/LimitOrdersConfirmModal/index.tsx
@@ -81,6 +81,7 @@ export function LimitOrdersConfirmModal(props: LimitOrdersConfirmModalProps) {
         priceImpact={priceImpact}
         buttonText={buttonText}
         recipient={recipient}
+        isPriceStatic={true}
       >
         <>
           <LimitOrdersDetails

diff --git a/apps/cowswap-frontend/src/modules/trade/pure/TradeConfirmation/index.tsx b/apps/cowswap-frontend/src/modules/trade/pure/TradeConfirmation/index.tsx
@@ -30,6 +30,7 @@ export interface TradeConfirmationProps {
   priceImpact: PriceImpact
   title: JSX.Element | string
   refreshInterval?: number
+  isPriceStatic?: boolean
   recipient: string | null
   buttonText?: React.ReactNode
   children?: JSX.Element
@@ -57,6 +58,7 @@ export function TradeConfirmation(props: TradeConfirmationProps) {
     buttonText = 'Confirm',
     children,
     recipient,
+    isPriceStatic,
   } = frozenProps || props
 
   /**
@@ -128,7 +130,7 @@ export function TradeConfirmation(props: TradeConfirmationProps) {
         {children}
         {/*Banners*/}
         {showRecipientWarning && <CustomRecipientWarningBanner orientation={BannerOrientation.Horizontal} />}
-        {isPriceChanged && <PriceUpdatedBanner onClick={resetPriceChanged} />}
+        {isPriceChanged && !isPriceStatic && <PriceUpdatedBanner onClick={resetPriceChanged} />}
         <ButtonPrimary onClick={handleConfirmClick} disabled={isButtonDisabled} buttonSize={ButtonSize.BIG}>
           {hasPendingTrade ? (
             <LongLoadText fontSize={15} fontWeight={500}>