-
Notifications
You must be signed in to change notification settings - Fork 454
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: update delegate example to include ipfs setup instructions #371
Conversation
docs: add the ability to add content via delegate routing
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.
LGTM
Co-Authored-By: dirkmc <dirkmdev@gmail.com>
I tried to follow instructions but I am getting error 404 when calling
Any Idea what I am missing? |
@lidel I started with a fresh setup locally and I am hitting issues now too. Going to look more into this, I'm assuming it's a configuration issue because my ipfs version has not changed. |
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.
@jacobheun fyi Gateway.APICommands
is not wired to anything in go-ipfs (ipfs/kubo#5565 (comment)), which blocks these docs from being useful.
I'm planning on giving this a wirl tomorrow - will let you know how it works out! |
@alanshaw but tomorrow never came. |
I'll never forget the look in Oli's eyes when I brought up that movie in the course of regular conversation. |
Yikes, we forgot about this one 🙈 |
@lidel this is quite outdated. We should close this as #507 (comment) is the new PR, which is also blocked |
K, let's continue in #507 (comment) |
This updates the delegate routing example to include instructions for setting up the go-ipfs node locally to support the needed API endpoints.
I also added an "Add Data" field to the UI and instructions to demonstrate content providing being delegable.