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.

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

Rename LeafData as IndexTreeLeafPreimage #3444

Closed
benesjan opened this issue Nov 28, 2023 · 0 comments · Fixed by #3794
Closed

Rename LeafData as IndexTreeLeafPreimage #3444

benesjan opened this issue Nov 28, 2023 · 0 comments · Fixed by #3794
Assignees

Comments

@benesjan
Copy link
Contributor

No description provided.

@github-project-automation github-project-automation bot moved this to Todo in A3 Nov 28, 2023
@benesjan benesjan self-assigned this Nov 28, 2023
benesjan added a commit that referenced this issue Jan 3, 2024
1. Removed redundant LeafData struct and used the one from protocol
circuits. I had to move the type to protocol-circuit/rollup-lib to
protocol-circuit/types to have it exposed.
2. Fixes #3444
3. Unified the naming in NullifierLeafPreimage in TS and Noir.
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Jan 3, 2024
Maddiaa0 pushed a commit that referenced this issue Jan 8, 2024
1. Removed redundant LeafData struct and used the one from protocol
circuits. I had to move the type to protocol-circuit/rollup-lib to
protocol-circuit/types to have it exposed.
2. Fixes #3444
3. Unified the naming in NullifierLeafPreimage in TS and Noir.
michaelelliot pushed a commit to Swoir/noir_rs that referenced this issue Feb 28, 2024
1. Removed redundant LeafData struct and used the one from protocol
circuits. I had to move the type to protocol-circuit/rollup-lib to
protocol-circuit/types to have it exposed.
2. Fixes AztecProtocol#3444
3. Unified the naming in NullifierLeafPreimage in TS and Noir.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant