-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore(deps): bump bdk wallet & friends to 1.0.0 #553
base: main
Are you sure you want to change the base?
Conversation
bdk_esplora = { version = "0.20.1", features = [ | ||
"async-https", | ||
"async-https-rustls", | ||
"tokio", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is needed now 😓 Cc @Zk2u
// F***! BDK 1.0 removed the `insert_tx` method, so we have to do this the hard way | ||
// Taken from https://github.com/bitcoindevkit/bdk/pull/1658/commits/3135e291d777d474ab4b76de36d43a96ff104a3c | ||
recovery_wallet.transactions().for_each(|tx| { | ||
l1w.insert_tx(tx.tx_node.tx); | ||
l1w.apply_update(Update { | ||
tx_update: TxUpdate { | ||
txs: vec![tx.tx_node.tx], | ||
..Default::default() | ||
}, | ||
..Default::default() | ||
}) | ||
.expect("could not insert transaction"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any ideas here @Zk2u?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC you can use the apply_unconfirmed_txs
.
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## main #553 +/- ##
==========================================
- Coverage 56.76% 56.75% -0.02%
==========================================
Files 306 306
Lines 31263 31264 +1
==========================================
- Hits 17747 17743 -4
- Misses 13516 13521 +5
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -76,8 +79,17 @@ pub async fn recover(args: RecoverArgs, seed: Seed, settings: Settings) { | |||
continue; | |||
} | |||
|
|||
// F***! BDK 1.0 removed the `insert_tx` method, so we have to do this the hard way |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there an issue filed on this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but as said above we can use method.apply_unconfirmed_txs
.
Just waiting input from @Zk2u since he did a lot of development as well with BDK in Strata.
Description
BDK 1.0.0 has been released. No more betas!
See: https://github.com/bitcoindevkit/bdk/releases/tag/wallet-1.0.0
Type of Change
Notes to Reviewers
Checklist