What's the point of emitting the block.timestamp in the create event?

[PaulRBerg]

Here:

flow/src/interfaces/ISablierFlow.sol

Line 40 in 76bcae3

uint40 lastTimeUpdate,

The lastUpdateTime can be inferred from the block timestamp of the block in which the create event was emitted. AFAIK, this property is available in the subgraph.

Asking you @sablier-labs/solidity and @razgraf (to double-check my understanding of the subgraph and whether this is a problem)?

[smol-ninja]

Agree that we can remove it.

[PaulRBerg]

Great, thanks for confirming. Waiting for @razgraf to confirm that this is OK.

[razgraf]

We have access, by default, to block timestamp, number etc. in the transaction metadata provided by indexers/subgraphs, so from our POV there's no need to store it.

[PaulRBerg]

OK great, thanks for confirming.