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.

Sign up for GitHub

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

Jump to bottom

Make the gov proposal options configurable #11057

Closed
4 tasks
robert-zaremba opened this issue Jan 28, 2022 · 2 comments · Fixed by #15151
Closed
4 tasks

Make the gov proposal options configurable #11057

robert-zaremba opened this issue Jan 28, 2022 · 2 comments · Fixed by #15151
Assignees
@tac0turtle
Labels
C:x/gov

Comments

@robert-zaremba
Copy link
Collaborator

Summary

In #11010 we removed deposit burnings when

  • unable to reach quorum
  • unable to reach deposit

Instead we let's make it configurable either using gov param or chain option (set during app wiring).

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@robert-zaremba robert-zaremba mentioned this issue Jan 28, 2022
Closed
4 tasks
@tac0turtle tac0turtle self-assigned this Jan 30, 2022
@cmwaters
Copy link
Contributor

What would be the default in this situation?

@tac0turtle
Copy link
Member

current behaviour would be default.

@tac0turtle tac0turtle mentioned this issue Jan 31, 2022
Closed
19 tasks
@tac0turtle tac0turtle moved this to 📝 Todo in Cosmos-SDK May 9, 2022
@tac0turtle tac0turtle moved this from 📝 Todo to 💪 In Progress in Cosmos-SDK May 9, 2022
@tac0turtle tac0turtle mentioned this issue Aug 4, 2022
Closed
@tac0turtle tac0turtle moved this from 💪 In Progress to 📝 Todo in Cosmos-SDK Jan 16, 2023
@tac0turtle tac0turtle mentioned this issue Feb 24, 2023
Merged
19 tasks
@github-project-automation github-project-automation bot moved this from Backlog to Done in Cosmos SDK Maintenance Feb 28, 2023
@github-project-automation github-project-automation bot moved this from 📝 Todo to 👏 Done in Cosmos-SDK Feb 28, 2023
@tac0turtle tac0turtle removed this from Cosmos-SDK May 31, 2023
@RollkitBot RollkitBot mentioned this issue Aug 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tac0turtle tac0turtle

Labels
C:x/gov
Projects
No open projects
Cosmos SDK Maintenance
Status: Done
Milestone
No milestone
4 participants
@robert-zaremba @tac0turtle @cmwaters @julienrbrt