Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

chainHead: Sync events with spec #14654

Merged
merged 6 commits into from
Jul 31, 2023
Merged

chainHead: Sync events with spec #14654

merged 6 commits into from
Jul 31, 2023

Conversation

lexnv
Copy link
Contributor

@lexnv lexnv commented Jul 27, 2023

The #14638 will handle the new error propagation of those methods and remove the old chainHead events.

Closes: #14637

// @paritytech/subxt-team

@lexnv lexnv added A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit B1-note_worthy Changes should be noted in the release notes T2-API This PR/Issue is related to APIs. labels Jul 27, 2023
@lexnv lexnv requested review from jsdw and skunert July 27, 2023 09:21
@lexnv lexnv self-assigned this Jul 27, 2023
@lexnv lexnv merged commit 115af33 into master Jul 31, 2023
5 checks passed
@lexnv lexnv deleted the lexnv/chainhead_sync_events branch July 31, 2023 07:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit T2-API This PR/Issue is related to APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RPC-Spec-V2] Sync chainHead events and reported errors with spec
3 participants