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

Remove ZkProgrammable in favor of new ZkProgram #198

Open
rpanic opened this issue Sep 10, 2024 · 1 comment · May be fixed by #256
Open

Remove ZkProgrammable in favor of new ZkProgram #198

rpanic opened this issue Sep 10, 2024 · 1 comment · May be fixed by #256
Assignees
@ejMina226

Comments

@rpanic
Copy link
Member

rpanic commented Sep 10, 2024

As of o1js 1.7.0, ZkPrograms accept a proofsEnabled parameter. Back when we created the ZkProgrammable utilities, the lack of that was the primary reason of that being created.
Now that has become unnecessary and should be removed.
@ejMina226 ejMina226 self-assigned this Dec 10, 2024
@ejMina226 ejMina226 moved this to In Progress in Main Board Dec 16, 2024
@ejMina226 ejMina226 linked a pull request Dec 19, 2024 that will close this issue
feature/remove-zkProgrammable #249
Closed
@ejMina226 ejMina226 linked a pull request Jan 17, 2025 that will close this issue
Remove zkProgrammable (rebased) WIP #256
Draft
@ejMina226 ejMina226 removed a link to a pull request Jan 17, 2025
feature/remove-zkProgrammable #249
Closed
@ejMina226
Copy link
Collaborator

See PR for status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ejMina226 ejMina226

Labels
None yet
Projects
Main Board
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove zkProgrammable (rebased) WIP proto-kit/framework
2 participants
@rpanic @ejMina226