Add Polimec HRMP info #5503
Add Polimec HRMP info #5503
Conversation
docs/build/build-hrmp-channels.md
Outdated
| The xcm-executor pallet can be forked by parachains' teams and modified to include logic to handle | ||
| [high-level HRMP instructions](https://paritytech.github.io/xcm-docs/journey/channels-and-bridges.html#hrmpnewchannelopenrequest). | ||
| These instructions are not included in the default version of the pallet, but can be used for | ||
| opening, accepting, and closing channels without relying on governance. |
There was a problem hiding this comment.
While this document gives concrete instructions on how to open HRMP channels between chains, this section falls short explaining what specific logic needs to be added to xcm-executor pallet to handle high-level HRMP instructions. Perhaps, a link to the forked code base and an actual project example that successfully opened channels without relying on governance would help.
There was a problem hiding this comment.
This suggestion came from my comments. We're open-sourcing the code at Polimec next Monday and could include it here. However, the codebase is still in development.
The wording is very good @ filippoweb3. It should be noted that the initial opening would probably not be done by a notification from the relay, since someone needs to send the hrmpInitOpenChannel extrinsic to the relay first for this to happen.
At Polimec we expect the other parachain to first initiate the opening from whatever means they want (probably sudo), and from this notification start our logic to check if we should accept and initiate the other direction channel.
There was a problem hiding this comment.
Perfect! Yeah, just want to ensure there are concrete instructions on this page and avoid ambiguity for parachain teams. Sorry for the mix-up with Referenda 340. We will link to your implementation once it is available.
There was a problem hiding this comment.
@JuaniRios thanks, we will leave this open until we can link to the instructions you will opensource :)
There was a problem hiding this comment.
@filippoweb3 @DrW3RK Sorry for the delay. We're open sourcing sometime this week, prob tomorrow
There was a problem hiding this comment.
@filippoweb3 our legal team is now reviewing the licences. ETA around Wednesday next week
There was a problem hiding this comment.
Our codebase is now open source!
Here are some interesting links you might want to use:
- polimec-node repo
- hrmp instruction implementation in xcm-executor fork
- impl of the new xcm-executor config types
Let me know if you have any questions :)
There was a problem hiding this comment.
@DrW3RK I added those links above, lmk if its ok
There was a problem hiding this comment.
Thank you @JuaniRios for contributing!
filippoweb3
added
A2 - Please Review
Co-authored-by: joe petrowski <[email protected]>
Co-authored-by: joe petrowski <[email protected]>
Co-authored-by: joe petrowski <[email protected]>
No description provided.