Merge pull request #870 from synonymdev/fix/cannot-send-over-balance

piotr-iohk · web-flow · commit 8c8e25e75a1c · 2026-03-26T17:50:40.000+01:00
fix: cap onchain send amount to balance
diff --git a/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt b/app/src/main/java/to/bitkit/viewmodels/AppViewModel.kt
@@ -1169,7 +1169,10 @@ class AppViewModel @Inject constructor(
                 }
             }
 
-            SendMethod.ONCHAIN -> amount > Defaults.dustLimit.toULong()
+            SendMethod.ONCHAIN -> {
+                val maxSendable = walletRepo.balanceState.value.maxSendOnchainSats
+                amount > Defaults.dustLimit.toULong() && amount <= maxSendable
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1169,7 +1169,10 @@ class AppViewModel @Inject constructor(`
`1169`	`1169`	`}`
`1170`	`1170`	`}`
`1171`	`1171`
`1172`		`- SendMethod.ONCHAIN -> amount > Defaults.dustLimit.toULong()`
	`1172`	`+ SendMethod.ONCHAIN -> {`
	`1173`	`+ val maxSendable = walletRepo.balanceState.value.maxSendOnchainSats`
	`1174`	`+ amount > Defaults.dustLimit.toULong() && amount <= maxSendable`
	`1175`	`+ }`
`1173`	`1176`	`}`
`1174`	`1177`	`}`
`1175`	`1178`