-
Notifications
You must be signed in to change notification settings - Fork 65
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
Update to bdk v0.26.0 #133
Update to bdk v0.26.0 #133
Conversation
Hey looks like you need do to a |
2e2e0e1
to
c29442d
Compare
Ah my bad.. Updated.. |
I updated bdk-reserves "update_bdk_026" branch with bdk rc.2. |
src/utils.rs
Outdated
@@ -390,6 +390,7 @@ pub(crate) fn new_blockchain( | |||
retry: wallet_opts.proxy_opts.retries, | |||
timeout: wallet_opts.electrum_opts.timeout, | |||
stop_gap: wallet_opts.electrum_opts.stop_gap, | |||
validate_domain: false, |
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 should be true since that's the default prior to adding the option.
validate_domain: false, | |
validate_domain: true, |
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.
ACK.. Missed that thought..
41c8ea5
to
8db5ef3
Compare
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.
ACK 8db5ef3
8db5ef3
to
8948fc1
Compare
8948fc1
to
52dd87e
Compare
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.
ACK 52dd87e
Description
Upgrade to bdk v0.26.0
Notes to the reviewers
Checklists
All Submissions:
cargo fmt
andcargo clippy
before committing