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

WarpSync: Show number of required peers in informant #14190

Merged
merged 1 commit into from
May 22, 2023

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented May 22, 2023

This makes it for the user more obvious on what we are waiting and not just "waiting for peers".

This makes it for the user more obvious on what we are waiting and not just "waiting for peers".
@bkchr bkchr added A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. B1-note_worthy Changes should be noted in the release notes T0-node This PR/Issue is related to the topic “node”. labels May 22, 2023
@bkchr bkchr requested a review from a team May 22, 2023 15:34
Copy link
Contributor

@lexnv lexnv left a comment

Choose a reason for hiding this comment

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

Amazing! Thanks for improving this!

@bkchr bkchr merged commit e0de101 into master May 22, 2023
@bkchr bkchr deleted the bkchr-warp-required-peers branch May 22, 2023 20:23
gpestana pushed a commit that referenced this pull request May 27, 2023
This makes it for the user more obvious on what we are waiting and not just "waiting for peers".
Ank4n pushed a commit that referenced this pull request Jul 8, 2023
This makes it for the user more obvious on what we are waiting and not just "waiting for peers".
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
This makes it for the user more obvious on what we are waiting and not just "waiting for peers".
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. T0-node This PR/Issue is related to the topic “node”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants