Skip to content

Add burnchain_op info to event emitter#216

Merged
kantai merged 3 commits intomasterfrom
feat/burn-events
Feb 21, 2023
Merged

Add burnchain_op info to event emitter#216
kantai merged 3 commits intomasterfrom
feat/burn-events

Conversation

@kantai
Copy link
Copy Markdown
Contributor

@kantai kantai commented Feb 17, 2023

Description

This adds more detailed event information for burnchain ops (in this case, L1 operations), using the burnchain_op field which was added to stacks-blockchain in stacks-network/stacks-core#3517.

I still need to update the event-dispatcher.md documentation with these changes.

Applicable issues

Checklist

  • Test coverage for new or modified code paths
  • Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)

@kantai kantai linked an issue Feb 17, 2023 that may be closed by this pull request
@obycode
Copy link
Copy Markdown
Contributor

obycode commented Feb 21, 2023

Can #168 be closed in favor of this PR?

@saralab saralab requested a review from obycode February 21, 2023 17:05
@kantai
Copy link
Copy Markdown
Contributor Author

kantai commented Feb 21, 2023

Okay, updated the latest from master and added docs -- this should be ready for review.

Also, yes, I'm closing #168 in favor of this PR (this PR is closer to what got implemented in the stacks-node as well).

Copy link
Copy Markdown
Contributor

@obycode obycode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@kantai kantai merged commit 422d2d9 into master Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add base layer transaction type for API

2 participants