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

refactor XCM emulator #2988

Open
xlc opened this issue Aug 9, 2023 · 2 comments
Open

refactor XCM emulator #2988

xlc opened this issue Aug 9, 2023 · 2 comments

Comments

@xlc
Copy link
Contributor

xlc commented Aug 9, 2023

Currently XCM emulator makes too much assumption about everything so that it is not generic and cannot be used for many commonly enough setups such as non u128 balance type or H160 account id type.

A large refactor is required to remove those assumptions and move all the code depends on pallet-balances to a separate optional extension. Not every chain have pallet-balances. Each chain may have different AccountId type and Balance type and they have to be configurable.

@NachoPal
Copy link
Contributor

@xlc Please take a look at #2958

Hopefully you'll see some of the improvements you were asking (removed Balance pallet dependancy for example)
You are right about the Account type , that can be improved.

@NachoPal
Copy link
Contributor

Also #2892 will come to improve and clean up dependancies.

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

No branches or pull requests

2 participants