remove vpn check (#1855)

Author: janndriessen

src/app/earn/components/earn-widget/components/smart-trade-button.tsx

@@ -56,7 +56,7 @@ export function SmartTradeButton(props: SmartTradeButtonProps) {
 
   const { chainId } = useNetwork()
   const { open } = useAppKit()
-  const { isRestrictedCountry, isUsingVpn } = useProtectionContext()
+  const { isRestrictedCountry } = useProtectionContext()
   const { slippage } = useSlippage()
 
   const {
@@ -99,27 +99,21 @@ export function SmartTradeButton(props: SmartTradeButtonProps) {
     ) {
       return [WarningType.restricted]
     }
-    if (
-      !isTradablePair &&
-      isUsingVpn &&
-      !hiddenWarnings?.includes(WarningType.vpn)
-    ) {
-      return [WarningType.vpn]
-    }
+    // if (
+    //   !isTradablePair &&
+    //   isUsingVpn &&
+    //   !hiddenWarnings?.includes(WarningType.vpn)
+    // ) {
+    //   return [WarningType.vpn]
+    // }
     if (slippage > 9 && !hiddenWarnings?.includes(WarningType.priceImpact)) {
       return [WarningType.priceImpact]
     }
     if (!hiddenWarnings?.includes(WarningType.flashbots)) {
       return [WarningType.flashbots]
     }
     return []
-  }, [
-    hiddenWarnings,
-    isRestrictedCountry,
-    isTradablePair,
-    isUsingVpn,
-    slippage,
-  ])
+  }, [hiddenWarnings, isRestrictedCountry, isTradablePair, slippage])
 
   const onClick = useCallback(async () => {
     if (buttonState === TradeButtonState.connectWallet) {

src/components/smart-trade-button/index.tsx

@@ -53,7 +53,7 @@ export function SmartTradeButton(props: SmartTradeButtonProps) {
 
   const { chainId } = useNetwork()
   const { open } = useAppKit()
-  const { isRestrictedCountry, isUsingVpn } = useProtectionContext()
+  const { isRestrictedCountry } = useProtectionContext()
   const { slippage } = useSlippage()
 
   const {
@@ -96,27 +96,21 @@ export function SmartTradeButton(props: SmartTradeButtonProps) {
     ) {
       return [WarningType.restricted]
     }
-    if (
-      !isTradablePair &&
-      isUsingVpn &&
-      !hiddenWarnings?.includes(WarningType.vpn)
-    ) {
-      return [WarningType.vpn]
-    }
+    // if (
+    //   !isTradablePair &&
+    //   isUsingVpn &&
+    //   !hiddenWarnings?.includes(WarningType.vpn)
+    // ) {
+    //   return [WarningType.vpn]
+    // }
     if (slippage > 9 && !hiddenWarnings?.includes(WarningType.priceImpact)) {
       return [WarningType.priceImpact]
     }
     if (!hiddenWarnings?.includes(WarningType.flashbots)) {
       return [WarningType.flashbots]
     }
     return []
-  }, [
-    hiddenWarnings,
-    isRestrictedCountry,
-    isTradablePair,
-    isUsingVpn,
-    slippage,
-  ])
+  }, [hiddenWarnings, isRestrictedCountry, isTradablePair, slippage])
 
   const onClick = useCallback(async () => {
     if (buttonState === TradeButtonState.connectWallet) {

src/lib/hooks/use-is-token-pair-tradable.ts

@@ -6,10 +6,9 @@ export function useIsTokenPairTradable(
   outputTokenSymbol: string,
   chainId: number,
 ): boolean {
-  const { isForbiddenAddress, isRestrictedCountry, isUsingVpn } =
-    useProtectionContext()
+  const { isForbiddenAddress, isRestrictedCountry } = useProtectionContext()
   if (isForbiddenAddress) return false
-  if (!isRestrictedCountry && !isUsingVpn) return true
+  if (!isRestrictedCountry) return true
   // When tokenlists is used everywhere, we can just pass these objects as function
   // arguments instead of the token symbol
   const outputToken = getTokenByChainAndSymbol(chainId, outputTokenSymbol)