-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expose the add_foreign_utxo
method on TxBuilder type
#358
Expose the add_foreign_utxo
method on TxBuilder type
#358
Conversation
I have tested and it works well inside Flutter. |
067e8ad
to
aa26634
Compare
aa26634
to
3c6bdcc
Compare
I had previously added the I could add it back if we think there is demand for it, but for now removing it seems like the simplest solution. |
3c6bdcc
to
6377c59
Compare
Hello, we have a use case for this. We need to sweep/drain funds from multiple wallets to a single destination address. The mobile app has the public descriptors for all wallets. I'm hoping to do this with a single transaction. It seemed like this method would be the one to use to combine psbts together but it's not exposed via ffi yet. |
Closing this as additional features will not be merged in the 0.X version of bdk-ffi. The new 1.0 API will not require this anymore (or at least not in the same form), so further work on this is not necessary. All efforts are currently going to the 1.0 release! 🚀 |
Description
This PR fixes #329.
Notes to the reviewers
Not ready for review yet.
Changelog notice
cargo fmt
andcargo clippy
before committingNew Features: