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

Fast General Admin Track for opening HRMP channels #2736

Merged
merged 7 commits into from Apr 10, 2024

Conversation

noandrea
Copy link
Collaborator

@noandrea noandrea commented Apr 3, 2024

What does it do?

Introduce a new origin HrmpOpenOrigin that is allowed to open a Hrmp channel.

On Moonbase this change limits the scope of opening the Hrmp channel to the Fast General Admin track, removing the possibility to make operation on the channel.

On Moonriver and Moonbeam this change adds the Fast General Admin track that allows to open a new Hrmp channel.

The Fast General Admin track is similar to the General Admin track but with less strict requirements

What important points reviewers should know?

Here is the forum post describing the change https://forum.moonbeam.network/t/new-track-for-opening-hrmp-xcm-channels/1669

@noandrea noandrea added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes labels Apr 3, 2024
@noandrea noandrea marked this pull request as draft April 3, 2024 13:41
@noandrea noandrea changed the title Fast Admin Track for opening HRMP channels Fast Genearl Admin Track for opening HRMP channels Apr 3, 2024
@noandrea noandrea changed the title Fast Genearl Admin Track for opening HRMP channels Fast General Admin Track for opening HRMP channels Apr 3, 2024
Copy link
Contributor

github-actions bot commented Apr 3, 2024

Coverage Report

@@                       Coverage Diff                       @@
##           master   noandrea-fast-admin-track-2      +/-   ##
===============================================================
+ Coverage   72.46%                        72.47%   +0.01%     
  Files         229                           229              
+ Lines       70512                         70523      +11     
===============================================================
+ Hits        51094                         51105      +11     
  Misses      19418                         19418              
Files Changed Coverage
/pallets/xcm-transactor/src/lib.rs 85.74% (+0.26%) 🔼

Coverage generated Tue Apr 9 09:10:56 UTC 2024

@noandrea noandrea force-pushed the noandrea-fast-admin-track-2 branch from 45ae0c9 to 61eb12e Compare April 8, 2024 08:33
@noandrea noandrea marked this pull request as ready for review April 8, 2024 08:34
Co-authored-by: Andrea Giacobino <no.andrea@gmail.com>
@noandrea noandrea merged commit 0430625 into master Apr 10, 2024
26 of 27 checks passed
@noandrea noandrea deleted the noandrea-fast-admin-track-2 branch April 10, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes D3-trivial PR contains trivial changes in a runtime directory that do not require an audit not-breaking Does not need to be mentioned in breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants