Merge pull request #158 from blocknative/fix/no-wallet

Show correct dialog when no wallet installed.

Closes #156

Author: lnbc1QWFyb24

src/interfaces.ts

@@ -76,7 +76,7 @@ export interface WalletModule {
   }>
   link?: string
   installMessage?: (wallets: {
-    currentWallet: string
+    currentWallet: string | undefined
     selectedWallet: string
   }) => string
   preferred?: boolean
@@ -96,7 +96,7 @@ export interface Helpers {
 }
 
 export interface WalletInterface {
-  name: string
+  name: string | undefined
   connect?: () => Promise<{ message: string } | undefined>
   disconnect?: () => void
   loading?: Promise<undefined>

src/modules/select/content.ts

@@ -1,5 +1,5 @@
 export const extensionInstallMessage = (helpers: {
-  currentWallet: string
+  currentWallet: string | undefined
   selectedWallet: string
 }) => {
   const { currentWallet, selectedWallet } = helpers
@@ -29,8 +29,7 @@ export const extensionInstallMessage = (helpers: {
   } else {
     return `
     <p style="font-size: 0.889rem; font-family: Helvetica Neue; margin: 0.889rem 0;">
-    You'll need to install <b>${selectedWallet}</b> to continue. Install
-    the extension and then
+    You'll need to install <b>${selectedWallet}</b> to continue. Once you have it installed, go ahead and
     <span
     class="bn-clickable"
       style="color: #4a90e2; font-size: 0.889rem; font-family: Helvetica Neue;"

src/utilities.ts

@@ -102,7 +102,7 @@ export function createModernProviderInterface(provider: any): WalletInterface {
             )
         }
       ),
-    name: getProviderName(provider) || 'unknown'
+    name: getProviderName(provider)
   }
 }
 
@@ -117,7 +117,7 @@ export function createLegacyProviderInterface(provider: any): WalletInterface {
     balance: {
       get: () => getBalance(provider)
     },
-    name: getProviderName(provider) || 'unknown'
+    name: getProviderName(provider)
   }
 }
 

src/views/WalletSelect.svelte

@@ -130,7 +130,7 @@
       installMessage =
         module.installMessage &&
         module.installMessage({
-          currentWallet: walletAlreadyInstalled || 'unknown',
+          currentWallet: walletAlreadyInstalled,
           selectedWallet: selectedWalletModule.name
         })