-
Notifications
You must be signed in to change notification settings - Fork 21
feat: zero-knowledge proof guide #1398
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
Conversation
|
Skipping AI review because this PR is from a fork. A maintainer can start the review by commenting /review in this PR. |
|
/review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update—across contract-dev/zero-knowledge.mdx, there are several suggestions to align structure, tone, and safety; please apply the inline suggestions.
Kaladin13
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great article, lets fix AI review results and my comments
|
@Kaladin13, I made corrections, you can check it. |
|
@mysteryon88 Lint fails, please run |
|
@Kaladin13 Fixed |
Kaladin13
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good!
verytactical
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've cleaned up this branch and pushed the result into the zk branch of this repo, because I don't have perms to push into the fork.
Please recommit it back from your account, so that your contribution is preserved in repo history.
|
@verytactical Did I do everything right? |
verytactical
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
^
|
@verytactical If I understand correctly, then it's ready. |
anton-trunov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
|
@mysteryon88 please help us fix formatting issues |
|
@anton-trunov run |
Yeah, just push those changes now, please :) UPD: Thanks! |
|
Sorry for making it a long trip: we've managed to catch up with all the PRs only now, and are concurrently on some other projects. Just a heads up for any future PRs: I recommend keeping the "let maintainers push into the branch" setting on when a PR is created. It'd take less review roundtrips, and we would likely fix all these minor issues on our own. |
|
@verytactical No worries at all — everything’s totally fine. I understand, and I didn’t know about that feature, but I’ll make sure to keep it enabled in future PRs. |

Closes #111
Add Zero-Knowledge Proofs Guide for TON
Summary
This PR adds a comprehensive guide for working with zero-knowledge proofs on the TON blockchain, covering the complete workflow from circuit creation to on-chain verification.
Changes
Documentation
contract-dev/zero-knowledge.mdxNavigation
contract-dev/zero-knowledgeto the "Contract development" section indocs.jsonDictionary Updates
resources/dictionaries/custom.txt)resources/dictionaries/two-letter-words-ban.txt)Quality Assurance
npm run check:spell- all spelling checks passed