Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Include runtime API defs into metadata #11648

Closed
xlc opened this issue Jun 12, 2022 · 7 comments
Closed

Include runtime API defs into metadata #11648

xlc opened this issue Jun 12, 2022 · 7 comments
Assignees
Labels
T1-runtime This PR/Issue is related to the topic “runtime”.

Comments

@xlc
Copy link
Contributor

xlc commented Jun 12, 2022

Context: polkadot-js/api#4930

This will help client to query the available runtime APIs and able to invoke them with correct data.

@athei
Copy link
Member

athei commented Jul 6, 2022

cc @ascjones

This should be added to the sp_api::decl_runtime_apis! proc macro. It is essentially the same that ink! does for its messages minus selector generation.

@gilescope
Copy link
Contributor

Given that we're pushing more people to use state Call on the grounds that runtime api calls are on-chain upgradable (unlike rpc calls), we really do need to include them in the metadata. It's currently not a great dev experience that people code them up and then don't see them in the polkadot-js when they run it against their new chain.

@athei
Copy link
Member

athei commented Jul 21, 2022

I agree. However, the same is true for RPCs. So it doesn't make it worse at least.

@bkchr
Copy link
Member

bkchr commented Jul 26, 2022

Yeah, you also only see available rpc calls, but that is all. You don't see parameters that you need to pass and that you get back.

@gilescope
Copy link
Contributor

gilescope commented Oct 11, 2022 via email

@agryaznov
Copy link
Contributor

addressed in #13302

@agryaznov agryaznov removed their assignment Feb 14, 2023
@lexnv lexnv self-assigned this Mar 15, 2023
@juangirini juangirini added the T1-runtime This PR/Issue is related to the topic “runtime”. label Jun 7, 2023
@lexnv
Copy link
Contributor

lexnv commented Jul 19, 2023

The PR which added RuntimeAPI support for the metadata

Stabilization PR for the Metadata V15 (features added to the V15 included in the PR description for more details):

@lexnv lexnv closed this as completed Jul 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
Status: Done
Development

No branches or pull requests

7 participants