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

docs: improve network usage instructions #941

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

mattstam
Copy link
Contributor

Clarify the instructions to using cast and getting whitelisted.

@@ -17,8 +17,8 @@ SP1_PROVER=network SP1_PRIVATE_KEY=... RUST_LOG=info cargo run --release

- `SP1_PROVER` should be set to `network` when using the prover network.

- `SP1_PRIVATE_KEY` should be set to your [private key](#key-setup). You will need
to be using a [permissioned](#get-access) key to use the network.
- `SP1_PRIVATE_KEY` should be set to your [private key](./setup.md#key-setup). You will need
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These links should be ./setup I think: https://docs.succinct.xyz/prover-network/setup

You can run mdbook locally to test, check readme

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested them, they're correct.

@@ -6,30 +6,43 @@ Succinct [has been building](https://blog.succinct.xyz/succinct-network/) the Su

## Get access

Currently the network is permissioned, so you need to gain access through Succinct. After you have completed the key setup below, you can submit your address in this [form](https://docs.google.com/forms/d/e/1FAIpQLSd-X9uH7G0bvXH_kjptnQtNil8L4dumrVPpFE4t8Ci1XT1GaQ/viewform) and we'll contact you shortly.
**[FILL OUT THIS FORM](https://forms.gle/rTUvhstS8PFfv9B3A)** to gain access to the Succinct
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO this was better as it was before

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ratankaliani said that it was too hard to find the form

book/prover-network/setup.md Outdated Show resolved Hide resolved
@mattstam mattstam requested a review from ctian1 June 18, 2024 18:17
@mattstam mattstam force-pushed the mattstam/docs-network-improve branch from f9f7c75 to 5696531 Compare June 18, 2024 18:56
@mattstam mattstam changed the base branch from dev to main June 18, 2024 18:57
@mattstam mattstam force-pushed the mattstam/docs-network-improve branch from 5696531 to ead6346 Compare June 18, 2024 18:58
@mattstam mattstam enabled auto-merge (squash) June 18, 2024 19:24
@mattstam mattstam force-pushed the mattstam/docs-network-improve branch from ead6346 to 04b03fa Compare June 18, 2024 21:19
@mattstam mattstam changed the base branch from main to dev June 18, 2024 21:20
@mattstam mattstam merged commit b0628f4 into dev Jun 18, 2024
7 of 12 checks passed
@mattstam mattstam deleted the mattstam/docs-network-improve branch June 18, 2024 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants