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

add total supply to token storage #338

Closed
tzemanovic opened this issue Aug 16, 2022 · 2 comments · Fixed by #714
Closed

add total supply to token storage #338

tzemanovic opened this issue Aug 16, 2022 · 2 comments · Fixed by #714
Assignees
Labels

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Aug 16, 2022

We should add the total supply to the token VP to avoid iteration to calculate inflation (the VP can simply reject any changes to it in a transaction as only the protocol can mint tokens). This is needed for inflation system, see #230 (review)

@cwgoes
Copy link
Collaborator

cwgoes commented Jan 13, 2023

@brentstone perhaps this is covered as part of the proof-of-stake inflation work?

@brentstone
Copy link
Collaborator

@brentstone perhaps this is covered as part of the proof-of-stake inflation work?

Yes, didn't notice there was an explicit issue for this. It is in #714.

@github-project-automation github-project-automation bot moved this from Todo to Tested in Devnet in Namada-Old Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Tested in Devnet
Development

Successfully merging a pull request may close this issue.

3 participants