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

burn deposit documentation update #1473

Merged
merged 2 commits into from
May 13, 2022
Merged

burn deposit documentation update #1473

merged 2 commits into from
May 13, 2022

Conversation

hxrts
Copy link
Contributor

@hxrts hxrts commented May 6, 2022

deposit burn behaviour was changed in cosmos/cosmos-sdk#11011

deposit burn behaviour was changed in cosmos/cosmos-sdk#11011
@codecov
Copy link

codecov bot commented May 6, 2022

Codecov Report

Merging #1473 (cfe725e) into main (4e07112) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1473   +/-   ##
=======================================
  Coverage   15.68%   15.68%           
=======================================
  Files          10       10           
  Lines        1371     1371           
=======================================
  Hits          215      215           
  Misses       1145     1145           
  Partials       11       11           

1. **Expire** - deposits will be burned if the deposit period ends before reaching the minimum deposit (64 ATOM)
2. **Fail** to reach quorum - deposits will be burned for proposals that do not reach quorum ie. 40% of all staked ATOM must vote
3. **Are vetoed** - deposits for proposals with 33.4% of voting power backing the 'NoWithVeto' option are also burned
Deposits are burned only when proposals are vetoed i.e. 33.4% of voting power backing the 'NoWithVeto' option.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense. see below

Copy link
Contributor Author

@hxrts hxrts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added link to gov module spec

docs/governance/process.md Outdated Show resolved Hide resolved
@okwme okwme merged commit 97a79c3 into main May 13, 2022
@okwme okwme deleted the hxrts-patch-1 branch May 13, 2022 11:20
tokamak-git pushed a commit that referenced this pull request Jun 21, 2022
* burn deposit documentation update

deposit burn behaviour was changed in cosmos/cosmos-sdk#11011

* Update docs/governance/process.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants