perf(wallet): limit transaction polling to local wallets

LN-Zap · Oct 10, 2020 · 69ca996 · 69ca996
1 parent c773c33
commit 69ca996
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/renderer/components/App/App.js b/renderer/components/App/App.js
@@ -28,6 +28,7 @@ const PEERS_REFETCH_BACKOFF_SCHEDULE = 2
 const appScheduler = createScheduler()
 
 const App = ({
+  activeWalletSettings,
   isAppReady,
   modals,
   redirectPayReq,
@@ -67,16 +68,18 @@ const App = ({
       maxDelay: PEERS_MAX_REFETCH_INTERVAL,
       backoff: PEERS_REFETCH_BACKOFF_SCHEDULE,
     })
-    appScheduler.addTask({
-      task: () => fetchTransactions(true),
-      taskId: 'fetchTransactions',
-      baseDelay: TX_REFETCH_INTERVAL,
-    })
     appScheduler.addTask({
       task: updateAutopilotNodeScores,
       taskId: 'updateAutopilotNodeScores',
       baseDelay: AUTOPILOT_SCORES_REFRESH_INTERVAL,
     })
+    if (activeWalletSettings.type === 'local') {
+      appScheduler.addTask({
+        task: () => fetchTransactions(true),
+        taskId: 'fetchTransactions',
+        baseDelay: TX_REFETCH_INTERVAL,
+      })
+    }
 
     // Set wallet open state.
     setIsWalletOpen(true)
@@ -96,6 +99,7 @@ const App = ({
       appScheduler.removeAllTasks()
     }
   }, [
+    activeWalletSettings,
     initActivityHistory,
     isSyncedToGraph,
     fetchDescribeNetwork,
@@ -155,6 +159,7 @@ const App = ({
 }
 
 App.propTypes = {
+  activeWalletSettings: PropTypes.object,
   fetchDescribeNetwork: PropTypes.func.isRequired,
   fetchPeers: PropTypes.func.isRequired,
   fetchSuggestedNodes: PropTypes.func.isRequired,

diff --git a/renderer/containers/App/App.js b/renderer/containers/App/App.js
@@ -2,7 +2,7 @@ import React from 'react'
 import { connect } from 'react-redux'
 import { fetchPeers } from 'reducers/peers'
 import { fetchDescribeNetwork } from 'reducers/network'
-import { setIsWalletOpen } from 'reducers/wallet'
+import { setIsWalletOpen, walletSelectors } from 'reducers/wallet'
 import { updateAutopilotNodeScores } from 'reducers/autopilot'
 import { initActivityHistory } from 'reducers/activity'
 import { fetchTransactions } from 'reducers/transaction'
@@ -23,6 +23,7 @@ import ModalStack from './ModalStack'
 import AppErrorBoundary from './ErrorBoundary'
 
 const mapStateToProps = state => ({
+  activeWalletSettings: walletSelectors.activeWalletSettings(state),
   isAppReady: appSelectors.isAppReady(state),
   isSyncedToGraph: infoSelectors.isSyncedToGraph(),
   redirectPayReq: state.pay.redirectPayReq,