Skip to content
This repository has been archived by the owner on Aug 19, 2020. It is now read-only.

Document RPC Calls #56

Open
wpank opened this issue May 4, 2020 · 4 comments
Open

Document RPC Calls #56

wpank opened this issue May 4, 2020 · 4 comments
Labels
K4 - Integration Info about infrastructure and integration.

Comments

@wpank
Copy link

wpank commented May 4, 2020

People looking through the documentation may notice that RPC documentation is missing from the knowledge base. I'm aware they are referenced here from the autogeneration, but at least referencing them somewhere in the knowledge base would be useful. (A step further would be giving examples of the inputs/outputs of these, as the above link doesn't make this too explicit and can be a point of confusion).

@joepetrowski
Copy link
Contributor

We discussed this recently in chat. There should probably be an autogenerated set of RPC reference docs that we publish somewhere. For the KB, I think RPC documentation should perhaps have a basics page, but the primary content should really be across multiple areas:

  • RPC Basics: basic curl examples, protocols supported, list of tools
  • Node management
  • Defining runtime RPC endpoints (e.g. TransactionPayment.queryInfo)
  • Using RPC for storage (already in storage documentation)
  • Decoding RPC data (already in SCALE)

@joepetrowski joepetrowski added the K4 - Integration Info about infrastructure and integration. label May 7, 2020
@JoshOrndorff
Copy link
Contributor

JoshOrndorff commented May 7, 2020

Defining runtime RPC endpoints (e.g. TransactionPayment.queryInfo)

For crosslinking purposes: https://substrate.dev/recipes/3-entrees/custom-rpc.html

Also, I recently had this conversation

Fred

runtime-specific RPCs was (and is) an anti-pattern

Me

Ah, I didn't realize that.

Fred

The thing is, maybe the framework isn't good enough to let people do things through the generic APIs, and people are resorting to runtime-specific RPCs because they have no better option then it's on us to provider a better option

@joepetrowski
Copy link
Contributor

Yeah, I agree with Fred here, especially given the recent weights issue where the host couldn't talk to the runtime. So we might not document "how to create it" but perhaps "how to troubleshoot it".

@danforbes
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
K4 - Integration Info about infrastructure and integration.
Projects
None yet
Development

No branches or pull requests

4 participants