test: Use 'mainnet' as networkName in onboard

chore: Add remoteConfig for tests fix: Declaration Fix tests chore: Update comment
5afe · Dec 14, 2021 · e3a00c7 · e3a00c7
1 parent 1f5d4b7
commit e3a00c7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/logic/safe/utils/mocks/remoteConfig.json b/src/logic/safe/utils/mocks/remoteConfig.json
@@ -114,6 +114,7 @@
       "features": [
         "CONTRACT_INTERACTION",
         "DOMAIN_LOOKUP",
+        "EIP1559",
         "ERC721",
         "SAFE_APPS",
         "SAFE_TX_GAS_OPTIONAL"

diff --git a/src/logic/wallets/onboard.ts b/src/logic/wallets/onboard.ts
@@ -6,6 +6,11 @@ import { setWeb3 } from './getWeb3'
 import { fetchProvider, removeProvider } from './store/actions'
 import transactionDataCheck from './transactionDataCheck'
 import { getSupportedWallets } from './utils/walletList'
+import { ChainId, CHAIN_ID } from 'src/config/chain.d'
+
+const NETWORK_NAMES: Record<ChainId, string> = {
+  [CHAIN_ID.ETHEREUM]: 'mainnet',
+}
 
 const getOnboardConfiguration = () => {
   let lastUsedAddress = ''
@@ -15,7 +20,8 @@ const getOnboardConfiguration = () => {
   return {
     networkId: parseInt(_getChainId(), 10),
     // Is it mandatory for Ledger to work to send network name in lowercase
-    networkName: getChainName().toLowerCase(),
+    // @FIXME: Move to CGW
+    networkName: NETWORK_NAMES[_getChainId()] || getChainName().toLowerCase(),
     subscriptions: {
       wallet: (wallet: Wallet) => {
         if (wallet.provider) {