-
Notifications
You must be signed in to change notification settings - Fork 11
update decentralization docs #66
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
WalkthroughThe documentation for the attestor contracts was updated by renaming a section heading and revising the descriptive text to clarify the purpose and functionality of the zkFetch example, reflecting its use of Reclaim's decentralized zkFetch variant and removing outdated references. Changes
Possibly related PRs
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
content/docs/attestor-decentralization/attestor-contracts.mdx (1)
85-92: Correct spelling and wording for clarityMinor language tweaks will improve readability:
-Alternatively, you can try out an e2e [example](https://github.com/reclaimprotocol/attestor-contracts/tree/main/examples/zkfetch) of Reclaim's decentralized zkFetch. In contrast to "classical" zkFetch, this variant performs on-chain task creation and returns mutiple proofs based on the contract-specified `requiredAttestors` limit. +Alternatively, you can try out an end-to-end [example](https://github.com/reclaimprotocol/attestor-contracts/tree/main/examples/zkfetch) of Reclaim's decentralized zkFetch. In contrast to the "classical" zkFetch, this variant performs on-chain task creation and returns multiple proofs based on the contract-specified `requiredAttestors` limit.-2. **Populate your `.env`, add you private key (without the `0x` prefix):** +2. **Populate your `.env`, add your private key (without the `0x` prefix):**This fixes the “mutiple” typo, replaces the abbreviation “e2e” with “end-to-end”, adjusts articles, and corrects “you” → “your”.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/docs/attestor-decentralization/attestor-contracts.mdx(1 hunks)
🧰 Additional context used
🪛 LanguageTool
content/docs/attestor-decentralization/attestor-contracts.mdx
[grammar] ~83-~83: Use proper spacing conventions.
Context: ... * node index.js ### zkFetch Example Alternatively, you can try out an e2e [e...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
[grammar] ~85-~85: Ensure spelling is correct.
Context: ...orms on-chain task creation and returns mutiple proofs based on the contract-specified ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_000051)
[grammar] ~85-~85: Use proper spacing conventions.
Context: ...act-specified requiredAttestors limit. 1. **Change directory into /examples/zkfetch ...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
Updated attestor decentralization docs to reflect latest zkFetch changes.
Summary by CodeRabbit