Skip to content

Beefy equivocation: add runtime API methods#4993

Merged
serban300 merged 7 commits intoparitytech:masterfrom
serban300:beefy-equivocation-runtime
Jul 23, 2024
Merged

Beefy equivocation: add runtime API methods#4993
serban300 merged 7 commits intoparitytech:masterfrom
serban300:beefy-equivocation-runtime

Conversation

@serban300
Copy link
Copy Markdown
Contributor

@serban300 serban300 commented Jul 10, 2024

Related to #4523

Add runtime API methods for:

  • generating the ancestry proof
  • submiting a fork voting report
  • submitting a future voting report

@serban300 serban300 added the T15-bridges This PR/Issue is related to bridges. label Jul 10, 2024
@serban300 serban300 requested a review from a team July 10, 2024 12:08
@serban300 serban300 self-assigned this Jul 10, 2024
@serban300 serban300 requested a review from acatangiu as a code owner July 10, 2024 12:08

impl<Header: HeaderT, Id: RuntimeAppPublic> ForkVotingProof<Header, Id, OpaqueValue> {
/// Try to decode the `AncestryProof`.
pub fn try_into<AncestryProof: Decode>(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@serban300 impl TryInto or TryFrom does not work here?

Copy link
Copy Markdown
Contributor Author

@serban300 serban300 Jul 23, 2024

Choose a reason for hiding this comment

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

I tried, but TryInto must return an Error and here it was better to return an Option.

@serban300 serban300 added this pull request to the merge queue Jul 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 23, 2024
@serban300 serban300 enabled auto-merge July 23, 2024 10:20
@serban300 serban300 added this pull request to the merge queue Jul 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 23, 2024
@serban300 serban300 added this pull request to the merge queue Jul 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 23, 2024
@serban300 serban300 added this pull request to the merge queue Jul 23, 2024
Merged via the queue into paritytech:master with commit 216e8fa Jul 23, 2024
@serban300 serban300 deleted the beefy-equivocation-runtime branch July 23, 2024 15:10
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
Related to paritytech#4523

Add runtime API methods for:
- generating the ancestry proof
- submiting a fork voting report
- submitting a future voting report
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T15-bridges This PR/Issue is related to bridges.

Projects

Status: Done
Status: Audited
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants