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

Introduce reserve-asset transfer extrinsic #3010

Merged
merged 2 commits into from
May 12, 2021

Conversation

gavofyork
Copy link
Member

@gavofyork gavofyork commented May 11, 2021

CC @xlc

@gavofyork gavofyork added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels May 11, 2021
@@ -1265,6 +1265,7 @@ impl pallet_xcm::Config for Runtime {
type XcmExecuteFilter = OnlyWithdrawTeleportForAccounts;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need to update this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The extrinsic will still work without any update. I decided to leave it alone for now since with the extrinsic it's unnecessary to add a filter.

@gavofyork gavofyork merged commit 0188254 into master May 12, 2021
@gavofyork gavofyork deleted the gav-xcm-xt-reserve-asset-xfer branch May 12, 2021 19:07
imstar15 pushed a commit to imstar15/polkadot that referenced this pull request Aug 25, 2021
* Companion for paritytech#3010 (Reserve Xfer extrinsic)

* cargo update -p xcm

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants