-
Notifications
You must be signed in to change notification settings - Fork 321
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
XCM to moonbeam #1183
XCM to moonbeam #1183
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, only code style and typos suggestions.
runtime/moonbeam/src/lib.rs
Outdated
/// This is the struct that will handle the revenue from xcm fees | ||
/// We do not burn anything because we want to mimic exactly what | ||
/// the sovereign account has | ||
pub type XcmFeesToAccount_ = XcmFeesToAccount< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code style: avoid leading underscore, refer to original type explicitely (and don't import it)
pub type XcmFeesToAccount_ = XcmFeesToAccount< | |
pub type XcmFeesToAccount = xcm_primitives::XcmFeesToAccount< |
(also change other occurencies of XcmFeesToAccount_
)
runtime/moonbeam/src/lib.rs
Outdated
} | ||
} | ||
|
||
/// The hooks we wantt to run in Maintenance Mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/// The hooks we wantt to run in Maintenance Mode | |
/// The hooks we want to run in Maintenance Mode |
What does it do?
What important points reviewers should know?
Is there something left for follow-up PRs?
What alternative implementations were considered?
Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?
What value does it bring to the blockchain users?