-
Notifications
You must be signed in to change notification settings - Fork 100
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
Fabo/2341 better validator updates #2370
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2370 +/- ##
===========================================
+ Coverage 96.42% 96.43% +0.01%
===========================================
Files 106 106
Lines 2376 2383 +7
Branches 129 129
===========================================
+ Hits 2291 2298 +7
Misses 65 65
Partials 20 20
|
Codecov Report
@@ Coverage Diff @@
## develop #2370 +/- ##
===========================================
+ Coverage 96.43% 96.43% +<.01%
===========================================
Files 106 106
Lines 2381 2387 +6
Branches 129 129
===========================================
+ Hits 2296 2302 +6
Misses 65 65
Partials 20 20
|
PENDING.md
Outdated
@@ -0,0 +1,2 @@ | |||
[Fixed] Better inplace updates to validators so the information (like signing info) never hides again once it was once loaded @faboweb |
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.
missing issue/PR number
|
||
// incrementally add the validator to the list or update it in place | ||
// "upsert": (computing, databases) An operation that inserts rows into a database table if they do not already exist, or updates them if they do. | ||
function upsertValidator(state, validator) { |
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.
nice addition ! this will be way faster with the object refactor
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.
utACK
Closes #2341
Description:
Thank you! 🚀
For contributor:
PENDING.md
with issue # and GitHub usernameFiles changed
in the github PR explorerFor reviewer: