Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: add total_mana_used to L2 block header #9716

Closed
just-mitch opened this issue Nov 4, 2024 · 0 comments · Fixed by #9868
Closed

feat: add total_mana_used to L2 block header #9716

just-mitch opened this issue Nov 4, 2024 · 0 comments · Fixed by #9868
Assignees
@LHerskind
Labels
C-l1-contracts Component: contracts deployed to L1 C-node Component: Aztec Node C-protocol-circuits Component: Protocol circuits (kernel & rollup)
Milestone
TestNet

Comments

@just-mitch
Copy link
Contributor

just-mitch commented Nov 4, 2024 •
edited by LHerskind
Loading

The total_mana_used values should be added to the L2 block header.
The value is to be populated similarly to how total_fees are done currently (accumulating the values from the levels below it). It should be fairly straightforward.
@just-mitch just-mitch added C-l1-contracts Component: contracts deployed to L1 C-node Component: Aztec Node C-protocol-circuits Component: Protocol circuits (kernel & rollup) labels Nov 4, 2024
@just-mitch just-mitch added this to the TestNet milestone Nov 4, 2024
@just-mitch just-mitch assigned just-mitch and LHerskind and unassigned just-mitch Nov 4, 2024
@just-mitch just-mitch changed the title Add total_mana_used to L2 block header feat: add total_mana_used to L2 block header Nov 7, 2024
This was referenced Nov 9, 2024
Merged
Closed
This was referenced Nov 20, 2024
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LHerskind LHerskind

Labels
C-l1-contracts Component: contracts deployed to L1 C-node Component: Aztec Node C-protocol-circuits Component: Protocol circuits (kernel & rollup)
Projects
None yet
Milestone
TestNet
Development

Successfully merging a pull request may close this issue.

feat: add total mana used to header AztecProtocol/aztec-packages
2 participants
@LHerskind @just-mitch