fix: Roi crash when full range selected

pancakeswap · Apr 22, 2023 · 05bb837 · 05bb837
1 parent 98a6dbe
commit 05bb837
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/apps/web/src/views/AddLiquidityV3/components/AprCalculator.tsx b/apps/web/src/views/AddLiquidityV3/components/AprCalculator.tsx
@@ -225,6 +225,7 @@ export function AprCalculator({
         volume24H={volume24H}
         priceUpper={priceUpper}
         priceLower={priceLower}
+        fullRangeSelected={formState.rightRangeTypedValue === true && formState.leftRangeTypedValue === true}
         priceSpan={priceSpan}
         onPriceSpanChange={setPriceSpan}
         onApply={onApply}

diff --git a/packages/uikit/src/widgets/RoiCalculator/RoiCalculator.tsx b/packages/uikit/src/widgets/RoiCalculator/RoiCalculator.tsx
@@ -52,6 +52,7 @@ export type RoiCalculatorProps = {
   price?: Price<Token, Token>;
   priceLower?: Price<Token, Token>;
   priceUpper?: Price<Token, Token>;
+  fullRangeSelected?: boolean;
   currencyAUsdPrice?: number;
   currencyBUsdPrice?: number;
   depositAmountInUsd?: number | string;
@@ -94,6 +95,7 @@ export function RoiCalculator({
   price,
   priceLower,
   priceUpper,
+  fullRangeSelected,
   volume24H,
   maxLabel,
   max,
@@ -175,6 +177,7 @@ export function RoiCalculator({
     quoteCurrency: currencyB,
     priceLower,
     priceUpper,
+    fullRangeSelected,
   });
   const { getDecrementLower, getIncrementLower, getDecrementUpper, getIncrementUpper } = useRangeHopCallbacks(
     currencyA,

diff --git a/packages/uikit/src/widgets/RoiCalculator/hooks/usePriceRange.ts b/packages/uikit/src/widgets/RoiCalculator/hooks/usePriceRange.ts
@@ -19,6 +19,7 @@ interface Params {
   quoteCurrency?: Currency;
   priceLower?: Price<Currency, Currency>;
   priceUpper?: Price<Currency, Currency>;
+  fullRangeSelected?: boolean;
 }
 
 interface PriceRangeInfo {
@@ -39,8 +40,9 @@ export function usePriceRange({
   feeAmount,
   priceLower: initialPriceLower,
   priceUpper: initialPriceUpper,
+  fullRangeSelected,
 }: Params): PriceRangeInfo | null {
-  const [fullRange, setFullRange] = useState(false);
+  const [fullRange, setFullRange] = useState(!!fullRangeSelected);
   const [priceLower, setPriceLower] = useState<Price<Currency, Currency> | undefined>(initialPriceLower);
   const [priceUpper, setPriceUpper] = useState<Price<Currency, Currency> | undefined>(initialPriceUpper);