src/modules/wallet/services/WalletWebSocketService.ts

@@ -289,21 +289,14 @@ export class WalletWebSocketService extends BaseService {
       this.walletService.addTransaction(payment)
     }
 
-    // Toast incoming payments only, and only once settled.
-    //
-    // This guard used to read `!payment.pending`, which is never present in
-    // the payload (a pydantic `@property`; see mapPaymentStatus in
-    // WalletService), so it was always true and an unsettled payment could
-    // announce itself as received. `status` is the real field.
-    //
-    // The old `else` branch toasted outgoing payments but was unreachable,
-    // since the enclosing condition already required `amount > 0`. It is
-    // dropped rather than fixed: a send made from this app is already
-    // confirmed by SendDialog's own toast, so re-announcing it here would
-    // double up.
-    if (payment.amount > 0 && payment.status === 'success') {
+    // Show toast notification for incoming payments
+    if (payment.amount > 0 && !payment.pending) {
       const amountSats = Math.abs(payment.amount / 1000)
+      if (payment.amount > 0) {
         this.toast.success(`Received ${amountSats} sats!`)
+      } else {
+        this.toast.info(`Sent ${amountSats} sats`)
+      }
     }
   }