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

Add pluggable BEEFY payload constructors #12428

Merged

Conversation

acatangiu
Copy link
Contributor

@acatangiu acatangiu commented Oct 5, 2022

Allows to configure the voter with custom PayloadProviders and thus support anything as BEEFY Payload.

Fixes paritytech/grandpa-bridge-gadget#409

polkadot companion: paritytech/polkadot#6112

@acatangiu acatangiu added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Oct 5, 2022
@acatangiu acatangiu self-assigned this Oct 5, 2022
primitives/beefy/src/payload.rs Outdated Show resolved Hide resolved
@acatangiu
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 3cb5a40 into paritytech:master Oct 6, 2022
@acatangiu acatangiu deleted the beefy-custom-payload branch December 13, 2022 08:39
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* primitives/beefy: move Payload to its own file

* primitives/beefy: add Payload tests

* primitives/beefy: add MmrRootProvider as custom BEEFY payload provider

* client/beefy: use generic BEEFY 'PayloadProvider'

* primitives/beefy: rename Payload::new to Payload::from_single_entry for clarity

* fix visibility

* fix cargo doc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow pluggable BEEFY payload constructors
3 participants