Steve Myers
Steve Myers
You should follow the PR template that should have been added to this PR automatically. In particular it helps reviewers to see a summary description of exactly what you're changing....
This is a small change but I think we should push it to the 2.0 milestone.
Closing since this address changes were fixed by #1402 and the can be fixed in another PR.
There's some more discussion for the original feature in bitcoindevkit/bdk#352.
I see how this is valuable especially since we had it in the pre-1.0.0 wallet API, so I agree this should stay in the 1.0.0-alpha milestone.
Since this can be enabled with a new feature flag without an api change I propose we move this to a post 1.0 milestone.
@LLFourn what if we use the RPC syncing as it is now but with a bitcoind node set to ["manual" pruning](https://github.com/bitcoin/bitcoin/pull/7871)? then we just have to add functionality to call...
@ikeogu yes feel free to work on this, but first look over the code and post here or ask on discord if you have any questions before you get started.
@srikanth-iyengar I think we need to figure out #1375 first, however that gets solved we should be able to use the same approach for this issue. What are your thoughts...
Yes it was @wiz who suggested we integrate with mempool's more extensive api, docs are here: https://mempool.space/api original issue is #289.