Skip to content

Comments

Make bdk_electrum and bdk_esplora support any checkpoint data type & update examples to use headers.#2107

Draft
evanlinjin wants to merge 3 commits intobitcoindevkit:masterfrom
evanlinjin:feature/electrum_cp_header
Draft

Make bdk_electrum and bdk_esplora support any checkpoint data type & update examples to use headers.#2107
evanlinjin wants to merge 3 commits intobitcoindevkit:masterfrom
evanlinjin:feature/electrum_cp_header

Conversation

@evanlinjin
Copy link
Member

Description

This is built on top of #2105 (rationale provided there). bdk_electrum and bdk_esplora changes should be in one PR as their changes are similar.

Notes to the reviewers

Changelog notice

Checklists

Progress:

  • bdk_electrum supports any checkpoint data
  • bdk_esplora supports any checkpoint data
  • Update examples to use headers

All Submissions:

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

This allows us to use any subset of a `Header` to construct checkpoint
data. Currently, only `BlockHash` and `Header` implement this.

Chain sources can bound the checkpoint data generic to this trait, so all
checkpoint data types that implement `FromBlockHeader` is supported by
the chain source.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api A breaking API change module-blockchain

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants