Skip to content
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

rpc: add rpc v2 chainSpec to polkadot #2859

Merged
merged 8 commits into from
Jan 8, 2024

Conversation

niklasad1
Copy link
Member

@niklasad1 niklasad1 commented Jan 5, 2024

The chainSpec RPC API from the v2 spec was only added to substrate-node and should be added to polkadot as well

/cc @lexnv

@niklasad1 niklasad1 requested a review from athei as a code owner January 5, 2024 14:50
@niklasad1 niklasad1 requested review from a team January 5, 2024 14:50
@niklasad1 niklasad1 requested a review from koute as a code owner January 5, 2024 14:50
@niklasad1 niklasad1 added the T3-RPC_API This PR/Issue is related to RPC APIs. label Jan 5, 2024
@niklasad1 niklasad1 added the R0-silent Changes should not be mentioned in any release notes label Jan 5, 2024
@niklasad1 niklasad1 requested a review from skunert January 5, 2024 15:02
@niklasad1 niklasad1 changed the title rpc: add rpc v2 chainSpec to polkadot and cumulus rpc: add rpc v2 chainSpec to polkadot Jan 5, 2024
@@ -99,6 +99,7 @@ pub fn create_full<C, P, SC, B>(
) -> Result<RpcExtension, Box<dyn std::error::Error + Send + Sync>>
where
C: ProvideRuntimeApi<Block>
+ sc_client_api::BlockBackend<Block>
Copy link
Contributor

@skunert skunert Jan 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I think we don't need this if you use the hash method from HeaderBackend

@niklasad1 niklasad1 merged commit a97a6f2 into master Jan 8, 2024
121 checks passed
@niklasad1 niklasad1 deleted the na-add-chainspec-v2-api-to-all-chains branch January 8, 2024 12:35
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
The [chainSpec RPC API from the v2
spec](https://paritytech.github.io/json-rpc-interface-spec/api/chainSpec.html)
was only added to substrate-node and should be added to polkadot as well

/cc @lexnv
bkchr pushed a commit that referenced this pull request Apr 10, 2024
* if bundled version is `None` and cli said "use bundled" => error

* update chain spec and transaction versions

* use generated runtime wrapper for PBH and KBH

* trigger CI

* Revert "trigger CI"

This reverts commit 7d9f51e34829d9224b352300d062e365243def5e.

* Revert "Revert "trigger CI""

This reverts commit c86ca0e264367e39abad29b9f09f09578a980568.

* Revert "Revert "Revert "trigger CI"""

This reverts commit 95e7da258bb2ace4ed2cafdb01833317f38cfcfc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T3-RPC_API This PR/Issue is related to RPC APIs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants