Merge pull request #672 from mrfelton/fix/invoice-notification

fix(invoice): only notify on settled invoices
LN-Zap · Aug 15, 2018 · 8457549 · 8457549
2 parents 1f20907 + b1be3f4
commit 8457549
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/app/reducers/invoice.js b/app/reducers/invoice.js
@@ -136,11 +136,13 @@ export const invoiceUpdate = (event, { invoice }) => dispatch => {
   // Fetch new balance
   dispatch(fetchBalance())
 
-  // HTML 5 desktop notification for the invoice update
-  const notifTitle = "You've been Zapped"
-  const notifBody = 'Congrats, someone just paid an invoice of yours'
+  if (invoice.settled) {
+    // HTML 5 desktop notification for the invoice update
+    const notifTitle = "You've been Zapped"
+    const notifBody = 'Congrats, someone just paid an invoice of yours'
 
-  showNotification(notifTitle, notifBody)
+    showNotification(notifTitle, notifBody)
+  }
 }
 // ------------------------------------
 // Action Handlers