fix: close wallet on terms reject, increase ledger poll

leather-io · Jan 14, 2021 · 9bc6100 · 9bc6100
1 parent c6816e6
commit 9bc6100
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 2 deletions.
diff --git a/app/hooks/use-ledger.ts b/app/hooks/use-ledger.ts
@@ -12,7 +12,7 @@ export enum LedgerConnectStep {
 }
 
 const SAFE_ASSUME_REAL_DEVICE_DISCONNECT_TIME = 1000;
-const POLL_LEDGER_INTERVAL = 250;
+const POLL_LEDGER_INTERVAL = 750;
 
 export function useLedger() {
   const [step, setStep] = useState(LedgerConnectStep.Disconnected);

diff --git a/app/main.dev.ts b/app/main.dev.ts
@@ -195,6 +195,8 @@ ipcMain.handle('reload-app', () => {
   mainWindow?.reload();
 });
 
+ipcMain.on('closeWallet', () => app.exit(0));
+
 //
 // TODO: refactor to be more generic
 // There's a bug where click handler doesn't fire for the top-level menu

diff --git a/app/pages/onboarding/00-terms/terms.tsx b/app/pages/onboarding/00-terms/terms.tsx
@@ -213,7 +213,7 @@ export const Terms: React.FC = () => {
       </Box>
       <Flex justifyContent="center" alignItems="center" mt="extra-loose" mb="120px">
         <Flex>
-          <OnboardingButton onClick={() => console.log('Close wallet')} mode="secondary">
+          <OnboardingButton onClick={() => api.closeWallet()} mode="secondary">
             Close
           </OnboardingButton>
           <OnboardingButton onClick={() => history.push(routes.WELCOME)} ml="base">

diff --git a/app/preload.ts b/app/preload.ts
@@ -93,6 +93,8 @@ const walletApi = {
   contextMenu: (menuItems: any) => ipcRenderer.send('context-menu-open', { menuItems }),
 
   installPath: () => ipcRenderer.sendSync('installPath'),
+
+  closeWallet: () => ipcRenderer.send('closeWallet'),
 };
 
 declare global {