Modify default wallets:

  * Separates default wallets into mobile and desktop
  * Moves Authereum to 2nd place
  * Adds isMobile parameter to determine which wallets to display
  * Adds deprecation warning for deprecated wallets

Author: taylorjdawson

src/modules/index.ts

@@ -13,9 +13,14 @@ const defaultWalletExplanation = `Wallets are used to send, receive, and store d
 export default function initializeModules(
   networkId: number,
   walletSelect: WalletSelectModuleOptions | undefined,
-  walletCheck: Array<WalletCheckModule | WalletCheckInit> | undefined
+  walletCheck: Array<WalletCheckModule | WalletCheckInit> | undefined,
+  isMobile: boolean
 ) {
-  const wallets = select(walletSelect && walletSelect.wallets, networkId)
+  const wallets = select(
+    walletSelect && walletSelect.wallets,
+    networkId,
+    isMobile
+  )
 
   return {
     walletSelect: {

src/modules/select/index.ts

@@ -3,33 +3,44 @@ import { isWalletInit } from '../../validation'
 
 // wallets that qualify for default wallets need to have no
 // init parameters that are required for full functionality
-const defaultWalletNames = [
+const desktopDefaultWalletNames = ['metamask', 'authereum', 'torus', 'opera']
+
+const mobileDefaultWalletNames = [
   'metamask',
-  'dapper',
+  'authereum',
   'coinbase',
   'trust',
-  'authereum',
   'torus',
   'opera',
   'operaTouch',
   'status',
   'hyperpay',
-  'unilogin',
   'tokenpocket',
   'dcent',
   'atoken'
 ]
 
 function select(
   wallets: Array<WalletInitOptions | WalletModule> | undefined,
-  networkId: number
+  networkId: number,
+  isMobile: boolean
 ) {
+  const defaultWalletNames = isMobile
+    ? mobileDefaultWalletNames
+    : desktopDefaultWalletNames
+
   if (wallets) {
     return Promise.all(
       wallets.map(wallet => {
         if (isWalletInit(wallet)) {
           const { walletName, ...initParams } = wallet
-          const module = getModule(walletName)
+
+          let module
+          try {
+            module = getModule(walletName)
+          } catch (error) {
+            console.warn(error)
+          }
 
           if (!module) {
             throw new Error(`${walletName} is not a valid walletName.`)
@@ -58,18 +69,19 @@ function select(
 
 function getModule(name: string): Promise<any> | undefined {
   switch (name) {
+    // Deprecated wallets
+    case 'dapper':
+    case 'squarelink':
+    case 'unilogin':
+      throw new Error(`${name.toUpperCase()} wallet has been deprecated`)
     case 'meetone':
       return import('./wallets/meetone')
     case 'metamask':
       return import('./wallets/metamask')
-    case 'dapper':
-      return import('./wallets/dapper')
     case 'portis':
       return import('./wallets/portis')
     case 'fortmatic':
       return import('./wallets/fortmatic')
-    case 'squarelink':
-      return import('./wallets/squarelink')
     case 'authereum':
       return import('./wallets/authereum')
     case 'trust':
@@ -96,8 +108,6 @@ function getModule(name: string): Promise<any> | undefined {
       return import('./wallets/wallet-link')
     case 'imToken':
       return import('./wallets/imtoken')
-    case 'unilogin':
-      return import('./wallets/unilogin')
     case 'mykey':
       return import('./wallets/mykey')
     case 'huobiwallet':

src/onboard.ts

@@ -58,7 +58,8 @@ function init(initialization: Initialization): API {
   const initializedModules = initializeModules(
     networkId,
     initialization.walletSelect,
-    initialization.walletCheck
+    initialization.walletCheck,
+    isMobile
   )
 
   let displayBranding: boolean