Skip to content
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

Allow MDMs to directly resolve the market #860

Closed
wants to merge 4 commits into from

Conversation

Chralt98
Copy link
Member

@Chralt98 Chralt98 commented Oct 31, 2022

No description provided.

@Chralt98 Chralt98 added the s:in-progress The pull requests is currently being worked on label Oct 31, 2022
@Chralt98 Chralt98 self-assigned this Oct 31, 2022
@Chralt98 Chralt98 changed the title Allow MDM's to directly resolve the market Allow MDMs to directly resolve the market Oct 31, 2022
@Chralt98 Chralt98 added s:on-hold Work on the pull request has been paused and removed s:in-progress The pull requests is currently being worked on labels Oct 31, 2022
@Chralt98
Copy link
Member Author

Because each MDM needs to resolve in the future, on_initialize is required for this. When we handle the timing in each MDM, we might end up with the situation, that we duplicate code (essentially the current handling of MarketIdsPerDisputeBlock in each MDM). So, I am in favour of using the current pm functionality and allow the MDM to control the MarketIdsPerDisputeBlock storage. #862

For authorized we have a correction period, which is much smaller than the dispute period. So we shouldn't really execute on_resolution with the extrinsic authorize_market_outcome, because there is the possibility, that the authority made a mistake.

@Chralt98 Chralt98 added s:abandoned This pull request is abandoned and removed s:on-hold Work on the pull request has been paused labels Dec 23, 2022
@Chralt98 Chralt98 closed this Dec 23, 2022
@Chralt98 Chralt98 deleted the chralt98-mdm-resolution branch December 23, 2022 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s:abandoned This pull request is abandoned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant