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

feat(silo-finance): exclude deprecated silos #6575

Merged

Conversation

JayWelsh
Copy link
Contributor

Adds a simple fallback mechanism to exclude deprecated silos.

Currently, TVL is being significantly (and incorrectly) inflated by the deprecated FARM silo:

Screenshot 2023-06-17 at 17 03 33

While the functionality to signal deprecations is being worked on for the subgraph, this will act as a fallback mechanism to avoid incorrect calculations.

@llamatester
Copy link

The adapter at projects/silo exports TVL:

arbitrum                  7.91 M
arbitrum-borrowed         5.52 M
ethereum-borrowed         7.00 M
ethereum                  30.43 M
borrowed                  12.52 M

total                    38.35 M 

@g1nt0ki g1nt0ki merged commit a5f13e9 into DefiLlama:main Jun 18, 2023
1 check failed
buchaoqun added a commit to buchaoqun/DefiLlama-Adapters that referenced this pull request Jun 19, 2023
* 'main' of github.com:buchaoqun/DefiLlama-Adapters: (440 commits)
  add usdc to level
  fix broken adapters
  add arbitrum chain to level
  add new wallet BTC binance
  add new wallet binance cex
  fix defichain loans
  add hallmark to defidollar
  change cardano api keys
  mark morpho doublecounted
  bugfix
  feat(silo): exclude deprecated silos (DefiLlama#6575)
  kommunitas: add arb stacking
  update crocswap to use graph
  temp fix lido
  pearlfi
  Additon of the Monolend project (DefiLlama#6571)
  Update Minswap adapter use new dex-addresses API (DefiLlama#6570)
  Orbiter (DefiLlama#6572)
  readd pawnfi
  update code for waterfallbsc
  ...

# Conflicts:
#	projects/solv-protocol/index.js
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