Skip to content

feat: add parent_burn_block_hash to /new_burn_block payload #6037

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
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

hstove
Copy link
Contributor

@hstove hstove commented Apr 25, 2025

This adds a new field, parent_burn_block_hash, to the event observer payload for /new_burn_block. This will be used in #6020 to efficiently find forked btc blocks.

@hstove hstove requested a review from a team as a code owner April 25, 2025 13:49
@hstove hstove moved this to Status: In Review in Stacks Core Eng Apr 25, 2025
@hstove hstove modified the milestones: 3.1.0.0.8, 3.1.0.0.9 Apr 25, 2025
@hstove hstove self-assigned this Apr 25, 2025
@aldur aldur requested a review from kantai April 25, 2025 14:48
@wileyj
Copy link
Collaborator

wileyj commented Apr 25, 2025

@hstove

error[E0050]: method `announce_burn_block` has 7 parameters but the declaration in trait `chainstate::coordinator::BlockEventDispatcher::announce_burn_block` has 8

should be an easy fix

@hstove
Copy link
Contributor Author

hstove commented Apr 25, 2025

Reviewers, just pushed the compilation error fix, sorry about that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Status: In Review
Development

Successfully merging this pull request may close these issues.

2 participants