Total Holdings is summing Vault and Strategy Holdings

[dudesahn]

On the yearn.finance/stats page, Total Holdings is summing Vault and Strategy Holdings, which is a misleading number (and not a useful metric), as Vault Holdings already includes everything in the Strategy.

For now, I would advocate for simply removing the "Total Holdings" box and only having three (centered) boxes at the top of the stats page.

[WrongNebula]

As an end user, Total holdings should be the TVL, it should be the most accurate TVL (compared to stats.finance and DeFi pulse or debanking)

[dudesahn]

@WrongNebula I agree totally. I would argue for including the v2/v3 yEarn contracts as well in "Total Holdings".

However, TVL in this case (if we're just looking at yVaults, which seems to be what this page is focused on) is the same thing as "Total Vault Holdings". If we add in the holdings in all of the yEarn contracts, then I think that would be a good "Total Holdings".