messari / subgraphs

Standardized subgraphs for blockchain data
MIT License
511 stars 240 forks source link

#bug; YA `VaultSnapshot` initializer issues #2256

Closed melotik closed 1 year ago

melotik commented 1 year ago

Describe the bug It is possible that a vaultSnapshot can be initialized and the TVL (and other fields) are set to 0, but then they are never updated to their proper value.

This is causing false 0s in snapshots.

To fix this:

dhruv-chauhan commented 1 year ago

@dmelotik I've done the change for the protocols you mentioned. Just confirming if we have to do this for other YA subgraphs too, or are these it for now?

melotik commented 1 year ago

yes, those are the only protocols I saw that had the issue @dhruv-chauhan