Skip to content

Conversation

@niklasad1
Copy link
Contributor

@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-no-crate-publish-required The change does not require any crates to be re-published. 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
) -> 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
@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-no-crate-publish-required The change does not require any crates to be re-published. T3-RPC_API This PR/Issue is related to RPC APIs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants