-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Remove accounts to store deposits within Governance #3628
Comments
4 tasks
@sunnya97 mind if I take a stab at this? |
@alexanderbez This requires fixing the bank module to track total supply instead of the staking module. Essentially redoing PR #2939 that was pushed post launch. However, that is probably out of scope for v0.34.0 @jackzampolin |
Ok, great. Thank you for the update. Removing this from v0.34 then as this is a non-trivial modification. |
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently we use auth accounts to store deposits within governance, this should be updated to use a running tally, similar to how staking tracks coins.
REF: #3567
CC @sunnya97
The text was updated successfully, but these errors were encountered: