Edit HRMP opening info #5494
Closed
Edit HRMP opening info #5494
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hey @lrazovic I updated the info about opening HRMP channels. Let me know if there is additional info to add. |
filippoweb3
added
A2 - Please Review
joepetrowski
requested changes
Jan 10, 2024
JuaniRios
reviewed
Jan 11, 2024
| Each parachain is to use its own governance process to execute this. The call will be executed on | ||
| the Relay Chain. | ||
| Each parachain is to use its governance process to execute this. The call will be executed on the | ||
| Relay Chain. |
There was a problem hiding this comment.
If it is to be made only through governance, then why do we have the xcm instruction HrmpNewChannelOpenRequest sent by the relay as a notification?
At Polimec we are using this to do automatic channel opening, without going through governance.
There was a problem hiding this comment.
Although we had to modify the xcm-executor because it had that instruction left empty, so maybe that change can happen once the executor adds an implementation for those instructions 🤔
Co-authored-by: Juan Ignacio Rios <[email protected]>
Co-authored-by: Juan Ignacio Rios <[email protected]>
Co-authored-by: Juan Ignacio Rios <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removed the info about using the
force_transferwithin the batchAll call, as now the track is General Admin (not Root), which does not have permissions.Added info about transferring funds deposit (20 DOT + existential deposit + fees) to the system parachain's sovereign account before opening the channel and doing the proposal.