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

Introduce scheduler and use it for the democracy dispatch queue #5412

Merged
merged 34 commits into from Apr 1, 2020

Conversation

gavofyork
Copy link
Member

@gavofyork gavofyork commented Mar 26, 2020

See paritytech/polkadot#942

Migration Note

This PR does not include a migration because it was not necessary for Kusama at the time. Downstream users might need to add a migration if migrating a runtime from a point previous to this PR to after this PR.

@gavofyork gavofyork added the A0-please_review Pull request needs code review. label Mar 26, 2020
@gavofyork gavofyork added this to the Polkadot milestone Mar 26, 2020
@gavofyork gavofyork mentioned this pull request Mar 26, 2020
@gavofyork gavofyork closed this Mar 26, 2020
@gavofyork gavofyork added this to Done in Runtime via automation Mar 26, 2020
@gavofyork gavofyork reopened this Mar 26, 2020
Runtime automation moved this from Done to In progress Mar 26, 2020
@gavofyork gavofyork closed this Mar 27, 2020
Runtime automation moved this from In progress to Done Mar 27, 2020
@gavofyork gavofyork reopened this Mar 27, 2020
Runtime automation moved this from Done to In progress Mar 27, 2020
Copy link
Contributor

@marcio-diaz marcio-diaz left a 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!

Co-Authored-By: Marcio Diaz <marcio.diaz@gmail.com>
@gavofyork gavofyork merged commit 2449d21 into master Apr 1, 2020
Runtime automation moved this from In progress to Done Apr 1, 2020
@gavofyork gavofyork deleted the gav-democracy-uses-scheduler branch April 1, 2020 13:52
lamafab pushed a commit to lamafab/substrate that referenced this pull request Jun 16, 2020
* Fixes for democracy using Scheduler

* Revert branch change
@apopiak
Copy link
Contributor

apopiak commented Jun 30, 2020

Note that this PR might require downstream projects to add a migration.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants