-
Notifications
You must be signed in to change notification settings - Fork 15
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: impose DelayedStakingGasFee when staking #175
feat: impose DelayedStakingGasFee when staking #175
Conversation
See tendermint#102 for details
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
Oh, we missed upgrading Swagger version on this PR, let me upgrade swagger on #176 |
} | ||
|
||
params := k.GetParams(ctx) |
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.
Sorry for additional review,
How about
if numStakingCoinDenoms > 0 {
params := k.GetParams(ctx)
ctx.GasMeter().ConsumeGas(sdk.Gas(numStakingCoinDenoms)*params.DelayedStakingGasFee, "DelayedStakingGasFee")
}
and could you add comments for a link or description of DelayedStakingGasFee
|
||
Since the farming module has adopted delayed staking, | ||
handling `MsgStake` does not immediately consumes gas. | ||
Instead, gas for this newly queued staking is consumed at the end of the epoch. | ||
`DelayedStakingGasFee` is to impose gas fee for this reason. |
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.
Could you describe it in more detail? We can commit on #178 continuously @hallazzang
Description
See #102.
closes: #102
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes