search

zeko-labs / zeko

Zeko: zk-Rollup for Mina, a succinct blockchain
https://zeko.io
Apache License 2.0
23 stars 3 forks source link

Time of transactions #219

Closed MartinOndejka closed 4 weeks ago

MartinOndejka commented 1 month ago

Adds time to the diff on da layer. Needed for analytics and metadata for explorer. Upgrades the da node api to v2.

Order of upgrades:

  1. da nodes
  2. sequencer
  3. archive relay
L-as commented 1 month ago

Why does the DA need to know this? This smells wrong.

MartinOndejka commented 1 month ago

Why does the DA need to know this? This smells wrong.

It's our only persistent storage.

L-as commented 1 month ago

I don't see how that means this is the correct solution.

What do you want to use the time information for?

L-as commented 3 weeks ago

Please don't merge without re-review.