-
Notifications
You must be signed in to change notification settings - Fork 0
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: document how to deploy the forest explorer on a clean CloudFlare account #86
Conversation
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.
- KV needs to be created manually in CF and modified in the wrangler file.
- There should be a mention of modifying the routes as well, no?
- It would be great to change
SECRET_WALLET
toSECRET_CALIBNET_WALLET
here or in a pre-requisite PR. - The pre-requisite of a paid CF plan must be listed - it's not possible to run this out of the box on a free account.
Your worker has access to the following bindings:
- Durable Objects:
- RATE_LIMITER: RateLimiter
✘ [ERROR] A request to the Cloudflare API (/accounts/fe2fac3a8b68edfa53dc37afa56e26f3/workers/scripts/forest-explorer) failed.
In order to use Durable Objects, you must switch to a paid plan at
https://dash.cloudflare.com/fe2fac3a8b68edfa53dc37afa56e26f3/workers/plans [code: 10084]
It's possible to work around this by removing the rate limiter, but the steps and the tradeoff should be mentioned.
5. Let's flesh out the steps with some examples.
Otherwise, I managed to deploy it fine on a free account. I'll create a docker file later to encompass the steps above + software pre-requisites so it's possible to one-line deploy. It would help if one would not have to modify the wrangler file and inject the account and others via environment variables.
04bf250
119971c
to
04bf250
Compare
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
@hanabi1224 @sudo-shashank could you please re-review? I added some more information. |
04bf250
to
521c48b
Compare
521c48b
to
bcaf8a1
Compare
bcaf8a1
to
64cbd88
Compare
64cbd88
to
16c7065
Compare
Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes #74
Other information and links
Change checklist
adheres to the team's
documentation standards,
(if possible),