Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Tokens subgraph: Update totalSupply if tokens minted or burned#79

Merged
0xGabi merged 2 commits intoaragon:masterfrom
fabriziovigevani:tokens-total-supply-fix
Jun 27, 2020
Merged

Tokens subgraph: Update totalSupply if tokens minted or burned#79
0xGabi merged 2 commits intoaragon:masterfrom
fabriziovigevani:tokens-total-supply-fix

Conversation

@fabriziovigevani
Copy link
Copy Markdown
Contributor

Follows the same logic as holder balances (calculates total supply at previous block from first tracked Transfer.)

Updates totalSupply when transfer corresponds to tokens minted or tokens burned.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 26, 2020

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   29.10%   29.10%           
=======================================
  Files          53       53           
  Lines         835      835           
  Branches      135      135           
=======================================
  Hits          243      243           
  Misses        592      592           
Flag Coverage Δ
#unittests 29.10% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77f6048...e7a23bd. Read the comment docs.

Copy link
Copy Markdown
Contributor

@0xGabi 0xGabi left a comment

Choose a reason for hiding this comment

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

Nice! 🙌

@0xGabi 0xGabi merged commit 67c3648 into aragon:master Jun 27, 2020
@fabriziovigevani fabriziovigevani deleted the tokens-total-supply-fix branch June 29, 2020 14:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants