From f1851d47a4d6e4147b6b6f9b8b93ba2f372fa723 Mon Sep 17 00:00:00 2001
From: Daniel Sanchez <daniel.sanchez@gnosis.pm>
Date: Tue, 1 Jun 2021 10:28:44 +0200
Subject: [PATCH] Fix crash with Coinbase Wallet after upgrade to latest
 version (#2363)

* Fix name changing in bnc-onboard Coinbase Wallet

* Add comment in wallet provider list to explain the use
---
 .../WalletIcon/icons/icon-coinbase.svg        | 35 +++----------------
 .../components/WalletIcon/icons/index.ts      |  2 +-
 src/logic/wallets/getWeb3.ts                  |  4 ++-
 3 files changed, 8 insertions(+), 33 deletions(-)

diff --git a/src/components/AppLayout/Header/components/WalletIcon/icons/icon-coinbase.svg b/src/components/AppLayout/Header/components/WalletIcon/icons/icon-coinbase.svg
index 7c6cb2e2..04739158 100644
--- a/src/components/AppLayout/Header/components/WalletIcon/icons/icon-coinbase.svg
+++ b/src/components/AppLayout/Header/components/WalletIcon/icons/icon-coinbase.svg
@@ -1,31 +1,4 @@
-<svg width="40" height="40" viewBox="0 0 383 383" fill="none" xmlns="http://www.w3.org/2000/svg">
-		<g clip-path="url(#clip0)">
-		<g filter="url(#filter0_dd)">
-			<path d="M0.998047 0.572266L382.78 0.572266V382.354H0.998047L0.998047 0.572266Z" fill="url(#paint0_linear)"/>
-			<path fill-rule="evenodd" clip-rule="evenodd" d="M59.1074 191.572C59.1074 264.966 118.605 324.463 191.998 324.463C265.392 324.463 324.889 264.966 324.889 191.572C324.889 118.179 265.392 58.6816 191.998 58.6816C118.605 58.6816 59.1074 118.179 59.1074 191.572ZM158.037 148.752C153.144 148.752 149.178 152.718 149.178 157.611V225.533C149.178 230.426 153.144 234.393 158.037 234.393H225.959C230.852 234.393 234.818 230.426 234.818 225.533V157.611C234.818 152.718 230.852 148.752 225.959 148.752H158.037Z" fill="white"/>
-		</g>
-		</g>
-		<defs>
-			<filter id="filter0_dd" x="-23.002" y="-7.42773" width="429.782" height="429.782" filterUnits="userSpaceOnUse" color-interpolation-filters="sRGB">
-				<feFlood flood-opacity="0" result="BackgroundImageFix"/>
-				<feColorMatrix in="SourceAlpha" type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 127 0"/>
-				<feOffset dy="16"/>
-				<feGaussianBlur stdDeviation="12"/>
-				<feColorMatrix type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0.06 0"/>
-				<feBlend mode="normal" in2="BackgroundImageFix" result="effect1_dropShadow"/>
-				<feColorMatrix in="SourceAlpha" type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 127 0"/>
-				<feOffset/>
-				<feGaussianBlur stdDeviation="4"/>
-				<feColorMatrix type="matrix" values="0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0.04 0"/>
-				<feBlend mode="normal" in2="effect1_dropShadow" result="effect2_dropShadow"/>
-				<feBlend mode="normal" in="SourceGraphic" in2="effect2_dropShadow" result="shape"/>
-			</filter>
-			<linearGradient id="paint0_linear" x1="191.889" y1="0.572266" x2="191.889" y2="382.354" gradientUnits="userSpaceOnUse">
-				<stop stop-color="#2E66F8"/>
-				<stop offset="1" stop-color="#124ADB"/>
-			</linearGradient>
-			<clipPath id="clip0">
-				<rect width="381.782" height="381.782" fill="white" transform="translate(0.998047 0.572266)"/>
-			</clipPath>
-		</defs>
-	</svg>
+<svg width="40" height="40" viewBox="0 0 40 40" fill="none" xmlns="http://www.w3.org/2000/svg">
+	<path d="M20 40C31.0457 40 40 31.0457 40 20C40 8.9543 31.0457 0 20 0C8.9543 0 0 8.9543 0 20C0 31.0457 8.9543 40 20 40Z" fill="#1652F0"/>
+	<path fill-rule="evenodd" clip-rule="evenodd" d="M5.45508 20.0006C5.45508 28.0338 11.9673 34.546 20.0006 34.546C28.0338 34.546 34.546 28.0338 34.546 20.0006C34.546 11.9673 28.0338 5.45508 20.0006 5.45508C11.9673 5.45508 5.45508 11.9673 5.45508 20.0006ZM17.3137 15.3145C16.2091 15.3145 15.3137 16.2099 15.3137 17.3145V22.6882C15.3137 23.7928 16.2091 24.6882 17.3137 24.6882H22.6874C23.792 24.6882 24.6874 23.7928 24.6874 22.6882V17.3145C24.6874 16.2099 23.792 15.3145 22.6874 15.3145H17.3137Z" fill="white"/>
+</svg>
\ No newline at end of file
diff --git a/src/components/AppLayout/Header/components/WalletIcon/icons/index.ts b/src/components/AppLayout/Header/components/WalletIcon/icons/index.ts
index 71642f95..dfef4337 100644
--- a/src/components/AppLayout/Header/components/WalletIcon/icons/index.ts
+++ b/src/components/AppLayout/Header/components/WalletIcon/icons/index.ts
@@ -65,7 +65,7 @@ const WALLET_ICONS: WalletObjectsProps<IconValue> = {
     src: operaIcon,
     height: 25,
   },
-  [WALLET_PROVIDER.WALLETLINK]: {
+  [WALLET_PROVIDER.COINBASE_WALLET]: {
     src: coinbaseIcon,
     height: 25,
   },
diff --git a/src/logic/wallets/getWeb3.ts b/src/logic/wallets/getWeb3.ts
index ffd5f007..ab1e8a5b 100644
--- a/src/logic/wallets/getWeb3.ts
+++ b/src/logic/wallets/getWeb3.ts
@@ -9,6 +9,7 @@ import { getRpcServiceUrl } from 'src/config'
 import { isValidCryptoDomainName } from 'src/logic/wallets/ethAddresses'
 import { getAddressFromUnstoppableDomain } from './utils/unstoppableDomains'
 
+// This providers have direct relation with name assigned in bnc-onboard configuration
 export const WALLET_PROVIDER = {
   SAFE: 'SAFE',
   METAMASK: 'METAMASK',
@@ -19,7 +20,8 @@ export const WALLET_PROVIDER = {
   SQUARELINK: 'SQUARELINK',
   WALLETCONNECT: 'WALLETCONNECT',
   OPERA: 'OPERA',
-  WALLETLINK: 'WALLETLINK',
+  // This is the provider for WALLET_LINK configuration on bnc-onboard
+  COINBASE_WALLET: 'COINBASE WALLET',
   AUTHEREUM: 'AUTHEREUM',
   LEDGER: 'LEDGER',
   TREZOR: 'TREZOR',