Skip to content

MultiLocationsFilter to additionally filter by asset #724

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

Open
sander2 opened this issue Apr 4, 2022 · 1 comment
Open

MultiLocationsFilter to additionally filter by asset #724

sander2 opened this issue Apr 4, 2022 · 1 comment

Comments

@sander2
Copy link
Contributor

sander2 commented Apr 4, 2022

#714 introduced a filter for xcm locations to the xtokens pallet. However, it checks only the multilocation in isolation, which means that each destination is either supported for all currencies, or that it's not supported at all. However, some destinations only make sense for certain currencies, e.g. sending the chain's native token to the relay chain probably does not make a lot of sense. So I thought it would make sense to change MultiLocationsFilter to take a tuple of a destination and an asset. Thoughts?

@xlc
Copy link
Member

xlc commented Apr 4, 2022

Sounds reasonable. cc @Dengjianping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants