Skip to content

Replace bdk_tmp_plan with miniscript planning api #139

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

Open
LLFourn opened this issue Aug 9, 2023 · 0 comments
Open

Replace bdk_tmp_plan with miniscript planning api #139

LLFourn opened this issue Aug 9, 2023 · 0 comments
Labels
new feature New feature or request

Comments

@LLFourn
Copy link
Contributor

LLFourn commented Aug 9, 2023

@danielabrozzoni looks like she's gotten rust-bitcoin/rust-miniscript#481 (comment) to a good place. It would be helpful if we could replace what we currently use for this role with that PR.

There was a very outdated attempt at doing this by @afilini here: https://github.com/LLFourn/bdk_core_staging/pull/66/files#diff-c14cb78dacd186a4264416678c250c834363aadab6cb8d1657709ffc970ca9ffR469

This has the important sign_with_keymap whose logic needs to go somewhere probably but maybe there might be some state of the art in rust-bitcoin or rust-miniscript to help with the signing part.

The goal of this PR is to make a github dependency on that PR and see how we can use it in the existing examples that use bdk_tmp_plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
Status: Todo
Development

No branches or pull requests

2 participants