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

User friendly show state command #811

Closed
wants to merge 7 commits into from
Closed

Conversation

Jim8y
Copy link
Contributor

@Jim8y Jim8y commented Aug 13, 2021

image

@erikzhang
Copy link
Member

What's the difference?

@Jim8y
Copy link
Contributor Author

Jim8y commented Aug 15, 2021

What's the difference?

@erikzhang I created this pr first, this one solved two major issues:

  • adding a block update progress, indicating the process of consensus (the dot in the second line). Otherwise, the screen looks frozen.
  • give the tag colors making the state more clear to check. (This can be replaced by User friendly cli console write system #812 )

After I committed this pr, I found that the whole cli outputs are not user-friendly. So I created #812 which focuses on the color of different level (information, warning, error) logs.

@Jim8y Jim8y closed this Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants