-
Notifications
You must be signed in to change notification settings - Fork 225
Display state of Grandpa #134
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.
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
Merged
Merged
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
1bb41eb
Make it clear that settings apply only to list view
cmichi 138b4af
Add Jdenticon
cmichi 8c26dc8
Add Grandpa consensus visualisation
cmichi 3c830af
Remove fade-in animation
cmichi c570cbc
Update packages and yarn.lock
cmichi e4de35c
Broadcast only delta of what changed
cmichi 1e824c7
Minor code improvements
cmichi a7ba531
Use NodeId instead of Address in first dimension
cmichi 672a33f
Refactoring and improving naming
cmichi e4b4916
Display boxes only after size has been detected
cmichi b2253e0
Fix cache
cmichi a3e9f06
Send consensus info on first subscribe
cmichi 4b5505b
Increase cache size
cmichi 5c48daa
Send deltas only if block in cache
cmichi 0623e5b
Adjust cache size
cmichi 064506a
Make cache sizes dependent
cmichi bde604a
Ensure authority caches are aligned
cmichi 5b590ba
Extract function
cmichi 062866a
Handle restarts on authority set changes properly
cmichi a569f54
Fix backfill mechanism
cmichi 003aace
Display only blocks since last authority set change
cmichi b3d3507
Handle authority set sent on connect
cmichi 03b3ae4
Introduce Authority type
cmichi fdc86c5
Handle corner case
cmichi b8d689b
Display placeholder if name not yet available
cmichi 0a0acce
Replace with camelCase
cmichi 23c254e
Replace with correct types
cmichi 44bf2a5
Replace grandpa icon
cmichi 3a4ea63
Merge branch 'master' into cmichi-display-state-of-consensus
maciejhirsz 3a6427a
Change consensus icon to cube (finalized block icon)
cmichi bf4d9ea
Upgrade dependencies
cmichi 7040545
Implement thin backend instead of thick
cmichi ca5d29b
Cleanup and minor improvements
cmichi 0bc61dc
Minor refactoring
cmichi 019d1f5
Extract common code into function
cmichi ed429c6
Switch module to class
cmichi 6fe140a
Remove unused code
cmichi 4e73e8f
Clean markup
cmichi 5e0dbdf
Remove unused code
cmichi 174eba4
Revert "Upgrade dependencies"
cmichi 0da0c44
Update polkadot-identicon in frontend
cmichi 64d08c3
Run yarn install
cmichi acd1218
Update react-measure to 2.3.0
cmichi f198278
Improve typing by introducing partial type
cmichi 5da7937
Reduce indexing operations
cmichi b02f72d
Shorten function
cmichi 12fd5c9
Shorten function
cmichi 770b10f
Introduce initialiseConsensusViewByRef
cmichi 0bafe84
Remove dead conditional branch
cmichi bce0576
Return consensusView ref from initialiseConsensusView
cmichi 44863a5
Handle consensusView ref returned from initialiseConsensusView
cmichi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is more FYI, but I think the telemetry allows nested JSON now without having to serialize things as strings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do I understand you right that we should be able to remove the
serdeserialization happening here?(I had tried it naively without looking any further into it and ran into
the trait 'slog::Value' is not implemented for 'std::vec::Vec<std::string::String>'.)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think that was fixed recently.