feat: added OKX wallet (#1270)

* feat: added okx wallet * fix: clean up
AstarNetwork · Apr 30, 2024 · 350fcdb · 350fcdb
1 parent b66387d
commit 350fcdb
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 7 deletions.
diff --git a/src/assets/img/icon_okx.png b/src/assets/img/icon_okx.png
diff --git a/src/config/wallets.ts b/src/config/wallets.ts
@@ -30,6 +30,7 @@ export enum SupportWallet {
   WalletConnect = 'wallet-connect',
   Dcent = 'DcentWallet',
   DcentEvm = 'DcentWalletEvm',
+  Okx = 'okxwallet',
 }
 
 export enum SupportMultisig {
@@ -64,6 +65,7 @@ export const WalletModalOption = {
   EnkryptNative: SupportWallet.EnkryptNative,
   Dcent: SupportWallet.Dcent,
   DcentEvm: SupportWallet.DcentEvm,
+  Okx: SupportWallet.Okx,
 };
 
 export const SubstrateWallets = [
@@ -327,6 +329,16 @@ export const supportEvmWalletObj = {
     isSupportMobileApp: true,
     ethExtension: 'ethereum',
   },
+  [SupportWallet.Okx]: {
+    img: require('/src/assets/img/icon_okx.png'),
+    name: 'OKX Wallet',
+    source: SupportWallet.Okx,
+    walletUrl: 'https://www.okx.com/web3',
+    guideUrl: 'https://www.okx.com/web3',
+    isSupportBrowserExtension: true,
+    isSupportMobileApp: false,
+    ethExtension: 'okxwallet',
+  },
 };
 
 export const supportAllWalletsObj = {

diff --git a/src/hooks/useConnectWallet.ts b/src/hooks/useConnectWallet.ts
@@ -128,18 +128,11 @@ export const useConnectWallet = () => {
       const accounts = await requestAccounts();
       accounts?.length && setCurrentEcdsaAccount(accounts[0]);
 
-      const chainId = getChainId(currentNetworkIdx.value);
-
       const provider = getEvmProvider(currentWallet);
       if (!provider) {
         return false;
       }
 
-      // Memo: Do not change the network for the Bridge page
-      if (currentRouter.value.name !== 'Bridge') {
-        isSetupNetwork && (await setupNetwork({ network: chainId, provider }));
-      }
-
       // If SubWallet return empty evm accounts, it required to switch to evm network and will request accounts again.
       // This setup will not require from version 0.4.8
       if (accounts?.length === 0 && currentWallet === SupportWallet.SubWalletEvm) {