Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

FeeManager that puts fees in a central account #2358

Closed
wants to merge 2 commits into from
Closed

FeeManager that puts fees in a central account #2358

wants to merge 2 commits into from

Conversation

gilescope
Copy link
Contributor

Similar to XcmFeesTo32ByteAccount, an implementation of FeeManager that routes XCM fees to a central account.

Associated issue: #2320

@paritytech-ci paritytech-ci requested review from a team March 21, 2023 17:57
@gilescope gilescope added A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit T6-XCM This PR/Issue is related to XCM. labels Mar 21, 2023
xlc
xlc reviewed Mar 21, 2023
View reviewed changes
primitives/utility/src/lib.rs Show resolved Hide resolved
primitives/utility/src/lib.rs Outdated Show resolved Hide resolved
@gilescope
Copy link
Contributor Author

This has been rolled into #2433

@gilescope gilescope closed this Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@joepetrowski joepetrowski joepetrowski left review comments

@bkontur bkontur bkontur left review comments

@xlc xlc xlc left review comments

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit T6-XCM This PR/Issue is related to XCM.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gilescope @xlc @bkontur @joepetrowski