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

Implementer's Guide: Demote HRMP channel management to regular dispatchables #1704

Closed
pepyakin opened this issue Sep 11, 2020 · 0 comments · Fixed by #1900
Closed

Implementer's Guide: Demote HRMP channel management to regular dispatchables #1704

pepyakin opened this issue Sep 11, 2020 · 0 comments · Fixed by #1900
Labels
S0-design PR/Issue is in the design stage
Milestone

Comments

@pepyakin
Copy link
Contributor

Demote the HRMP channel management upward message kinds to regular entrypoints.

First reaction is that these entrypoints should go into the Router module.

I think it might also imply changing ValidationData structs. The reason is the following. Currently if the collator provides invalid data to the PVF and then PVF does something nonsensical then the candidate is rejected and collator misses out. With dispatchables the candidate becomes valid and we need to pass the required data through the ground truth (if any) to inform the PVF. On the other hand, we may want to remove some stuff from the TransientValidationData.

Related #1702

@pepyakin pepyakin added the S0-design PR/Issue is in the design stage label Sep 11, 2020
@pepyakin pepyakin added this to the Grosvenor milestone Sep 11, 2020
@pepyakin pepyakin mentioned this issue Sep 17, 2020
2 tasks
@pepyakin pepyakin mentioned this issue Nov 2, 2020
3 tasks
@ghost ghost closed this as completed in #1900 Nov 6, 2020
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
S0-design PR/Issue is in the design stage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant