You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I ran into an issue while working on our wallet integration that I think is related to this thread.
When I try to create a PSBT on testnet or signet, I keep getting this error:
Error: Insufficient funds: 0 sat available of 11 sat needed
I understand why this check is important on mainnet (to more easily catch and handle low balance situations). But when I'm just trying to test stuff out on other networks, it just gets in the way.
What do you think about disabling this specific error for any network besides Network::Bitcoin? That way, we could generate PSBTs and test wallet compatibility more easily on testnets without this error getting in the way all the time. Maybe we could log a warning or print a message about the low balance instead?
Let me know your thoughts! I'm happy to submit a PR if you think this change would be helpful.
Describe the bug
Both
coin_selection::Error
andCreateTxError
enums have an InsufficientFunds member. It is quite confusion proneExpected behavior
We shouldn't have duplicated enums
Proposed solution
keep only
coin_selection::Error
's memberThe text was updated successfully, but these errors were encountered: