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

module: consensus-param module #12652

Closed
2 of 4 tasks
Tracked by #10514
tac0turtle opened this issue Jul 20, 2022 · 1 comment · Fixed by #12905
Closed
2 of 4 tasks
Tracked by #10514

module: consensus-param module #12652

tac0turtle opened this issue Jul 20, 2022 · 1 comment · Fixed by #12905
Assignees

Comments

@tac0turtle
Copy link
Member

Summary

With the deprecation of the param module, there is a need to allow governance or another account to modify these parameters. In a few discussions with the sdk team we opted to create a consensus param module to house these parameters.

Theses parameters will need to be made available to base app so that the parameters can be sent back to tendermint when they are modified. We do something similar with the upgrade module and the version setter.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@alexanderbez
Copy link
Contributor

@marbar3778 let's be sure to reference this/add it to the EPIC.

@tac0turtle tac0turtle moved this to 📝 Todo in Cosmos-SDK Jul 26, 2022
@tac0turtle tac0turtle self-assigned this Aug 8, 2022
@tac0turtle tac0turtle moved this from 📝 Todo to 💪 In Progress in Cosmos-SDK Aug 8, 2022
@tac0turtle tac0turtle mentioned this issue Aug 11, 2022
19 tasks
Repository owner moved this from 💪 In Progress to 👏 Done in Cosmos-SDK Oct 5, 2022
@tac0turtle tac0turtle removed this from Cosmos-SDK Oct 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants