-
Notifications
You must be signed in to change notification settings - Fork 815
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pallet society to support Block Number Provider #6623
Update pallet society to support Block Number Provider #6623
Conversation
can you help review this too @gui1117 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
prdoc/pr_6623.prdoc
Outdated
- audience: Runtime Dev | ||
description: | | ||
This PR makes the block number provider used in the society pallet configurable. Before this PR, society pallet always used the system block number, | ||
with this PR some runtime can opt to use the relay chain block number instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the PRs with block provider, I would try to leave a comment for a user that to keep everything as is they just need to set it to System
. With just the current description I would still want to understand why, and what should I set.
…_block_number_provider
fddb6a2
/tip small |
@ggwpez A referendum for a small (20 DOT) tip was successfully submitted for @dharjeezy (12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW on polkadot). |
The referendum has appeared on Polkassembly. |
This PR introduces BlockNumberProvider config for pallet society.
closes part of #6297
Polkadot address: 12GyGD3QhT4i2JJpNzvMf96sxxBLWymz4RdGCxRH5Rj5agKW