Skip to content
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

Modify distribution module on consumer #90

Closed
mpoke opened this issue May 6, 2022 · 0 comments · Fixed by #130
Closed

Modify distribution module on consumer #90

mpoke opened this issue May 6, 2022 · 0 comments · Fixed by #130
Assignees

Comments

@mpoke
Copy link
Contributor

mpoke commented May 6, 2022

This issue is related to the "democracy module" issue (#66) and come up in the discussion here #66 (comment), i.e.,

the distribution module on the consumer needs to be modified to not distribute rewards (inflated native tokens) based on LastCommitInfo from Tendermint, but rather based on the stake from the staking module. In other words, in the BeginBlock of the distribution module, all stakers of native tokens should receive rewards (i.e., as if everyone has a vote in LastCommitInfo) and there is no special reward for block proposers.

@mpoke mpoke added type: feature-request New feature or request improvement scope: cosmos-sdk Integration with Cosmos SDK ccv labels May 6, 2022
@mpoke mpoke moved this to Todo in Replicated Security May 6, 2022
@mpoke mpoke added product and removed scope: cosmos-sdk Integration with Cosmos SDK ccv labels May 10, 2022
@mpoke mpoke moved this from Todo to In Progress in Replicated Security May 14, 2022
@mpoke mpoke added gov-cc and removed type: feature-request New feature or request improvement labels Jun 8, 2022
@mpoke mpoke moved this from In Progress to Waiting for review in Replicated Security Jul 8, 2022
@mpoke mpoke moved this from Waiting for review to In Progress in Replicated Security Jul 8, 2022
Repository owner moved this from In Progress to Done in Replicated Security Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants