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
Move posts' const to chain params #204
Conversation
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.
re-reviewed
added migrate `v0.8.0`
Codecov Report
@@ Coverage Diff @@
## master #204 +/- ##
==========================================
+ Coverage 83.23% 83.76% +0.53%
==========================================
Files 75 78 +3
Lines 2815 2902 +87
==========================================
+ Hits 2343 2431 +88
+ Misses 407 403 -4
- Partials 65 68 +3 |
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.
The overall implementation looks good. There are some things I would change but otherwise it's ready to be merged 💯
added migrate test to profiles
…rams' into leonardo/post-consts-to-chain-params
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.
LGTM
Description
This PR move posts' const into chain params in order to let them be changed through proposal and queried easily from clients.
Close #172 .
Close #205 .
Checklist
CHANGELOG.md
file.Files changed
in the Github PR explorer.