search

polkadot-fellows / xcm-format

Polkadot Cross Consensus-system Message format.
Apache License 2.0
182 stars 43 forks source link

add hrmp messages #18

Closed xlc closed 3 years ago

xlc commented 3 years ago

Copied message from https://github.com/paritytech/polkadot/blob/cb95b9a3cd3d3217bc911f97227d927c26f75dba/xcm/src/v0/mod.rs#L162-L205

Add new one to allow parachain to perform HRMP related actions without using Transact

Closes #9

cc @pepyakin @shaunxw

I would like to get those in before I add new messages to avoid conflicts in message type index

xlc commented 3 years ago

@gavofyork Do we need to worry bit more about grouping message types?

Right now we have three kind of messages: Instruction, Notification and Query. We have few categories of messages: Assets, Relay, HRMP, and Staking / Governance / Storage in future addition.

I am worried it will become bit messy if we just simply adding messages types in the sequential order. For example, if we want to add new top level Assets related type, it won't be placed next to other Assets related types.

gavofyork commented 3 years ago

V2 includes correct HRMP messages.