-
Notifications
You must be signed in to change notification settings - Fork 252
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
feat: add param for gas per message byte #965
feat: add param for gas per message byte #965
Conversation
Codecov Report
@@ Coverage Diff @@
## main #965 +/- ##
==========================================
- Coverage 29.74% 27.08% -2.67%
==========================================
Files 72 81 +9
Lines 8199 9067 +868
==========================================
+ Hits 2439 2456 +17
- Misses 5531 6375 +844
- Partials 229 236 +7
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I guess this PR depends on this one #893, right? |
I know we'll merge #893 soon, but converting to a draft until then as this PR has many duplicate changes and shouldn't be reviewed or merged until its predecessor is |
f344603
to
6059435
Compare
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.
Left an error message fix.
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.
left one comment that could be addressed later, otherwise LGTM
Change for removing const and using param in followup PR that addresses #1001 |
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 LGTM
Adds `gasPerMsgByte` as params to the payment module. Defines relevant stores and queries Relevant changes in `proto/payment/*` `x/payment/keeper*` `x/payment/types/*` - [x] finished 1st part of celestiaorg#949 Note: To be merged after celestiaorg#893 and constants for default values can be defined in a cumulative PR
Adds `gasPerMsgByte` as params to the payment module. Defines relevant stores and queries Relevant changes in `proto/payment/*` `x/payment/keeper*` `x/payment/types/*` - [x] finished 1st part of celestiaorg#949 Note: To be merged after celestiaorg#893 and constants for default values can be defined in a cumulative PR
Adds
gasPerMsgByte
as params to the payment module.Defines relevant stores and queries
Relevant changes in
proto/payment/*
x/payment/keeper*
x/payment/types/*
GasPerMsgByte
as a parameter to payment module #949Note: To be merged after #893 and constants for default values can be defined in a cumulative PR