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

PoolTogether V5 Update #10143

Closed
wants to merge 1 commit into from
Closed

PoolTogether V5 Update #10143

wants to merge 1 commit into from

Conversation

Ncookiez
Copy link
Contributor

@Ncookiez Ncookiez commented May 8, 2024

Name (to be shown on DefiLlama):

PoolTogether / PoolTogether V3 / PoolTogether V4 / PoolTogether V5

Twitter Link:

https://twitter.com/PoolTogether_

List of audit links if any:

https://docs.pooltogether.com/security/audits

Website Link:

https://pooltogether.com/

Methodology (what is being counted as tvl, how is tvl being calculated):

This PR simply adds more vault factories to the current script to calculate PoolTogether V5 TVL. Since the last update we've had more prize pool deployments with different vault factories. No ABI changes were necessary for this script.

The other changes are simply cleanup to avoid repeated code on each version's adapter, as well as including more relevant timestamps for the different V5 launches in hallmarks.

@llamatester
Copy link

The adapter at projects/pooltogether-v4 exports TVL:

polygon                   1.54 M
optimism                  783.99 k
ethereum                  315.62 k
avax                      55.31 k

total                    2.70 M 

@llamatester
Copy link

The adapter at projects/pooltogether-v5 exports TVL:

optimism                  2.71 M

total                    2.71 M 

@llamatester
Copy link

The adapter at projects/pooltogether exports TVL:

ethereum                  2.49 M
polygon                   81.13 k
celo                      41.85 k
bsc                       7.31 k

total                    2.62 M 

@g1nt0ki
Copy link
Member

g1nt0ki commented May 9, 2024

hi @Ncookiez thanks for the PR, included your changes here: a226f34

skipped the hallmarks change, it might crowd the tvl chart.

@g1nt0ki g1nt0ki closed this May 9, 2024
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