-
Notifications
You must be signed in to change notification settings - Fork 11
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
fix: support parachains without assets pallet in runtime #323
fix: support parachains without assets pallet in runtime #323
Conversation
add integeration tests for transferMultiAsset
add unit tests for getXcAssetIdByAssetId
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.
I'll have a closer look at this on Monday. But it looks super solid.
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, really solid!
I really like the integrationTests abstraction. Makes things way more organized and easier to work through! Just one small question, but good to merge once the conflict is resolved
…d assets pallet validity prior to returning early for parachains that do not support the assets pallet
fix conflicts refactor ParaToRelay integration tests
ParaTo*
direction integration tests and separate into different filesxTokens
beneficiaries forParaToPara
directiontransferMultiAsset*
totransferMultiasset*
ParaTo*
examplescloses: #320