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

Add babe randomness to relay state proof #1083

Merged

Conversation

4meta5
Copy link
Contributor

@4meta5 4meta5 commented Mar 14, 2022

cc @bkchr This is steps 2-4 for #463 (comment) :

  1. Then you need to add the keys here:
    relevant_keys.extend(ingress_channels.into_iter().map(|sender| {
  2. Then the relevant data is part of the relay chain proof that is send to the Parachain runtime.
  3. We probably need then some functionality to access this data. I think there is currently no such functionality outside of the parachain system.

Better to store the entire proof for now with all its values. As I said, I think that doesn't exist yet.

@bkchr bkchr added A0-pleasereview B0-silent Changes should not be mentioned in any release notes labels Mar 17, 2022
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

Ty

@bkchr bkchr merged commit 680c573 into paritytech:master Mar 17, 2022
@4meta5 4meta5 deleted the amar-add-babe-randomness-well-known-keys branch March 17, 2022 13:07
@JoshOrndorff
Copy link
Contributor

Thanks @4meta5 !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants