From 82dc658c92a66d40cf1d804370c2f2a2b77a1f87 Mon Sep 17 00:00:00 2001
From: Pedro Yves Fracari <yves.fracari@gmail.com>
Date: Tue, 11 Jun 2024 14:18:49 -0300
Subject: [PATCH] refactor: separete thresholds into constants file

---
 .../new/(components)/CreateAMMForm.tsx        |  3 ++-
 .../src/components/DepositForm.tsx            | 23 +++++++++++--------
 apps/cow-amm-deployer/src/lib/constants.ts    |  6 +++++
 3 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/apps/cow-amm-deployer/src/app/[userId]/new/(components)/CreateAMMForm.tsx b/apps/cow-amm-deployer/src/app/[userId]/new/(components)/CreateAMMForm.tsx
index 825ef3c2d..dd8524f6f 100644
--- a/apps/cow-amm-deployer/src/app/[userId]/new/(components)/CreateAMMForm.tsx
+++ b/apps/cow-amm-deployer/src/app/[userId]/new/(components)/CreateAMMForm.tsx
@@ -23,6 +23,7 @@ import { Form } from "#/components/ui/form";
 import { useManagedTransaction } from "#/hooks/tx-manager/useManagedTransaction";
 import { useDebounce } from "#/hooks/useDebounce";
 import { ConstantProductFactoryABI } from "#/lib/abis/ConstantProductFactory";
+import { UNBALANCED_USD_DIFF_THRESHOLD } from "#/lib/constants";
 import { COW_CONSTANT_PRODUCT_FACTORY } from "#/lib/contracts";
 import { IToken } from "#/lib/fetchAmmData";
 import { ammFormSchema } from "#/lib/schema";
@@ -227,7 +228,7 @@ export function CreateAMMForm({ userId }: { userId: string }) {
       <span>
         {amountUsdDiff} {debouncedAmountUsdDiff}
       </span>
-      {(debouncedAmountUsdDiff || 0) > 5000 && (
+      {(debouncedAmountUsdDiff || 0) > UNBALANCED_USD_DIFF_THRESHOLD && (
         <AlertCard title="Unbalanced amounts" style="warning">
           <p>
             The difference between the USD value of the two token amounts is
diff --git a/apps/cow-amm-deployer/src/components/DepositForm.tsx b/apps/cow-amm-deployer/src/components/DepositForm.tsx
index ad885d64e..193b6e833 100644
--- a/apps/cow-amm-deployer/src/components/DepositForm.tsx
+++ b/apps/cow-amm-deployer/src/components/DepositForm.tsx
@@ -10,6 +10,10 @@ import { TokenInfo } from "#/components/TokenInfo";
 import { Form, FormMessage } from "#/components/ui/form";
 import { useManagedTransaction } from "#/hooks/tx-manager/useManagedTransaction";
 import { useDebounce } from "#/hooks/useDebounce";
+import {
+  PRICE_IMPACT_THRESHOLD,
+  USD_VALUE_FOR_PRICE_IMPACT_WARNING,
+} from "#/lib/constants";
 import { ICowAmm } from "#/lib/fetchAmmData";
 import { calculatePriceImpact } from "#/lib/priceImpact";
 import { getDepositSchema } from "#/lib/schema";
@@ -124,15 +128,16 @@ export function DepositForm({
           </FormMessage>
         )
       }
-      {debouncedDepositUsdValue > 5000 && debouncedPriceImpact > 0.1 && (
-        <AlertCard style="warning" title="High Price Impact">
-          <p>
-            The price impact of this deposit is{" "}
-            {formatNumber(debouncedPriceImpact * 100, 2)}%. Deposits with high
-            price impact may result in lost funds.
-          </p>
-        </AlertCard>
-      )}
+      {debouncedDepositUsdValue > USD_VALUE_FOR_PRICE_IMPACT_WARNING &&
+        debouncedPriceImpact > PRICE_IMPACT_THRESHOLD && (
+          <AlertCard style="warning" title="High Price Impact">
+            <p>
+              The price impact of this deposit is{" "}
+              {formatNumber(debouncedPriceImpact * 100, 2)}%. Deposits with high
+              price impact may result in lost funds.
+            </p>
+          </AlertCard>
+        )}
 
       <Button
         loading={
diff --git a/apps/cow-amm-deployer/src/lib/constants.ts b/apps/cow-amm-deployer/src/lib/constants.ts
index 8cb991da2..32c09f847 100644
--- a/apps/cow-amm-deployer/src/lib/constants.ts
+++ b/apps/cow-amm-deployer/src/lib/constants.ts
@@ -1,2 +1,8 @@
 export const BLEU_APP_DATA =
   "0x4d821ddc9d656177dad4d5c2f76a4bff2ed514ff69fa4aa4fd869d6e98d55c89";
+
+export const PRICE_IMPACT_THRESHOLD = 0.05;
+
+export const UNBALANCED_USD_DIFF_THRESHOLD = 5000;
+
+export const USD_VALUE_FOR_PRICE_IMPACT_WARNING = 5000;