Results 340 comments of Dan Gould

[bitcoind-json-rpc-regtest](https://crates.io/crates/bitcoind-json-rpc-regtest) appears to be the favored upgrade path in the rust-bitcoin issue linked in OP

some functions we use from `Client` are missing. These are the ones we used and what's already implemented in bitcoind-json-rpc-regtest. - [x] generate_to_address - [x] get_new_address - [x] get_balances -...

Noting this isn't really so much tech debt as it is maintenance. When the dependency was taken on it was state of the art, and the state of the art...

Seems like [`corepc-node`](https://crates.io/crates/corepc-node) and [bitcoind-async-client](https://crates.io/crates/bitcoind-async-client/versions) would let us close this upgrade, test taproot, and close this issue for 1.0 stability. This is not really an API issue but it does...

Whenever this gets merged I suggest it get squashed rather than have all $n$ commits used.

I believe all of these are set to sane defaults to simplify UI as this is a simple spending wallet. Defaults allow the experience to be more reliable but as...

Until BDK allows txo locking (https://github.com/bitcoindevkit/bdk_wallet/issues/166), this would need to be done on the bullbitcoin mobile side

I'm not sure exactly how locking / persisting locks / spending unbroadcasted utxos should work, but I can reiterate [the constraints that caused me to raise this issue](https://github.com/bitcoindevkit/bdk/pull/913#issuecomment-2032662636). In summary,...

> I just want to see the in-process payjoin txn like I see unconfirmed txns... Yes this is the spirit of #641. "Detection" is showing that it's relevant, but perhaps...