-
Notifications
You must be signed in to change notification settings - Fork 327
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
Fix Companion docs #1207
Comments
This is tagged under "Fix companion docs" but this is bigger than docs right? We're also talking about removing code here. Your call, but this seems like its own issue providing some justification (linking to js-ipfs deprecation issue) with PR of code removal doc changes linked to it. I think we should do this before we ship MV3. |
Instead of "Set to "External" to connect to a local node using the HTTP API." how about 'Set to "External" to connect to a local Kubo node using the Kubo RPC API." Anytime we see "HTTP API" we should be fixing to disambiguate between the HTTP Gateway API and the Kubo RPC API. |
Other things regarding "API" in the settings screen API → Kubo RPC API |
@whizzzkid: I want to make sure you saw my comments that were added after merge: #1230 (review) |
@BigLep yes, I'll create a follow up later tonight. |
Fix: https://docs.ipfs.tech/how-to/companion-node-types/#external
The text was updated successfully, but these errors were encountered: