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

Superfluid TVL calculation fix #12396

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

cincauhangus
Copy link

Hello, I work with Superfluid team to update the TVL calculation for the Superfluid platform. These are the changes made:

  • Consolidating 'vesting TVL' into a single TVL as there is no vesting in Superfluid
  • Added new chains
  • Removed whitelisted & blacklisted Supertokens in favor of isListed flag from subgraph

@llamatester
Copy link

The adapter at projects/superfluid.js exports TVL:

optimism                  45.63 M
base                      4.00 M
polygon                   2.72 M
ethereum                  763.53 k
arbitrum                  118.02 k
xdai                      81.52 k
bsc                       536
avax                      468
scroll                    0

total                    53.31 M 

Copy link
Member

@g1nt0ki g1nt0ki left a comment

Choose a reason for hiding this comment

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

hi @cincauhangus thanks for the PR
It is mostly good except for removal of vesting, we have this as a hack, else it wont be fair comparison between streaming protocols.

The reason, we are doing this is, we cant really tell if the token being streamed is already in circulation or not, thats why to be safe, we count all tokens other than gas tokens & stablecoins under vesting

We apply the same rule for all streaming protocols, you can verify it in our codebase, search for streamingHelper without this check, streamflow tvl would be over 1B

@g1nt0ki g1nt0ki self-assigned this Nov 22, 2024
@g1nt0ki
Copy link
Member

g1nt0ki commented Dec 20, 2024

bump

@cincauhangus
Copy link
Author

Sorry, will send a fix ASAP

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