search

superfluid-finance / protocol-monorepo

Superfluid Protocol Monorepo: the specification, implementations, peripherals and development kits.
https://www.superfluid.finance
Other
875 stars 240 forks source link

[SUBGRAPH][FEATURE] Inconsistency: AccountTokenSnapshot has `updatedAt(Timestamp/BlockNumber)` but not the equivalent for "created at" #1907

Closed kasparkallas closed 8 months ago

kasparkallas commented 8 months ago

What? AccountTokenSnapshot has updatedAt(Timestamp/BlockNumber) but not the equivalent for "created at". Every other entity has both "created at" and "updated at", this is the only known exception to me. Seems like an oversight.

Why? I actually have use for the createdAtBlockNumber value when implementing time-travelling in the visualizer.

How? Add the values in the getOrInitAccountTokenSnapshot.