Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix api-derive subscribeFinalizedBlocks #5787

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix api-derive subscribeFinalizedBlocks #5787

wants to merge 1 commit into from

Conversation

XY-Wang
Copy link

@XY-Wang XY-Wang commented Jan 29, 2024

The api-derive subscribeFinalizedBlocks was using switchMap which causes it to only emit the latest block when the subscribeFinalizedHeads observable emits a range of headers. Replacing the switchMap with concatMap to emit all blocks while preserving order.

@XY-Wang
fix api-derive subscribeFinalizedBlocks
a7f8b01 
only emits latest block from the block range
@TarikGul TarikGul added P4 - Needs Investigation Requires analysis to determine cause or feasibility. Not fully understood, needs research first. and removed needs investigation labels Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
P4 - Needs Investigation Requires analysis to determine cause or feasibility. Not fully understood, needs research first.
Projects
Polkadot-js general project board
Status: P4 - Needs Investigation
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@XY-Wang @TarikGul