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

set DisputesHandler in initializer on Rococo #3788

Merged
1 commit merged into from
Sep 5, 2021
Merged

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Sep 5, 2021

It was previously set to () and the DisputesHandler is the primary way for the initializer & inherent handler to communicate info to disputes. Without this, the runtime will drop all disputes.

@rphmeier rphmeier 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 Sep 5, 2021
@rphmeier
Copy link
Contributor Author

rphmeier commented Sep 5, 2021

bot merge

@ghost
Copy link

ghost commented Sep 5, 2021

Waiting for commit status.

@ghost ghost merged commit d6d5f06 into master Sep 5, 2021
@ghost ghost deleted the rh-really-enable-disputes branch September 5, 2021 16:58
@@ -585,7 +585,7 @@ impl runtime_parachains::inclusion::RewardValidators for RewardValidators {

impl parachains_inclusion::Config for Runtime {
type Event = Event;
type DisputesHandler = ();
type DisputesHandler = ParasDisputes;
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be feature gated as well?

ordian added a commit that referenced this pull request Sep 6, 2021
* master:
  Add tests and modify as_vec implementation (#3715)
  Bump structopt from 0.3.22 to 0.3.23 (#3770)
  bump substrate and beefy (#3789)
  set `DisputesHandler` in initializer (#3788)
  Bump futures from 0.3.16 to 0.3.17 (#3742)
  Convert rococo chainspec to raw chainspec (#3785)
  feat/overseer: introduce closure init (#3775)
  comment out bridges runtime modules (broken) and update Rococo chain-spec (#3780)
  Add VoteLocking config (#3734)
  Enable disputes on rococo (#3764)
  XCM: Automatic Version Negotiation (#3736)
This pull request was closed.
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.

3 participants