-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Migrate Validators v0.36 #4770
Migrate Validators v0.36 #4770
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4770 +/- ##
=======================================
Coverage 53.88% 53.88%
=======================================
Files 270 270
Lines 17230 17230
=======================================
Hits 9285 9285
Misses 7261 7261
Partials 684 684 |
Rate sdk.Dec `json:"rate"` | ||
MaxRate sdk.Dec `json:"max_rate"` | ||
MaxChangeRate sdk.Dec `json:"max_change_rate"` | ||
Rate sdk.Dec `json:"rate"` |
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 what the struct actually looks like 👌
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.
ACK cause I know @alexanderbez tested this on mainnet w/ an export.
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.
ack
Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added a relevant changelog entry:
clog add [section] [-t <tag>] [-m <msg>]
Re-reviewed
Files changed
in the github PR explorerFor Admin Use: