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

When querying an existing block, use the same version of Runtime that was used to construct it. #11764

Conversation

RGafiyatullin
Copy link
Contributor

Addresses paritytech/polkadot-sdk#64 .

This is a draft for what is referred "solution-3" in the original issue.

Please do not use the comments-section of this PR to discuss the choice of solution-3 over any other alternative.
If you'd like to discuss the choice of "solution-3" — please consider posting your comments in the original issue.

@RGafiyatullin RGafiyatullin added A3-in_progress Pull request is in progress. No review needed at this stage. B3-apinoteworthy C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Jun 30, 2022
@RGafiyatullin RGafiyatullin self-assigned this Jun 30, 2022
@stale
Copy link

stale bot commented Jul 30, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Jul 30, 2022
@stale stale bot closed this Aug 14, 2022
@louismerlin louismerlin removed the D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. label Oct 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. C1-low PR touches the given topic and has a low impact on builders.
Projects
Status: done
Development

Successfully merging this pull request may close these issues.

2 participants