cumulus icon indicating copy to clipboard operation
cumulus copied to clipboard

refactor XCM emulator

Open xlc opened this issue 2 years ago • 2 comments

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.

xlc avatar Aug 09 '23 03:08 xlc

@xlc Please take a look at https://github.com/paritytech/cumulus/pull/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 avatar Aug 11 '23 12:08 NachoPal

Also https://github.com/paritytech/cumulus/issues/2892 will come to improve and clean up dependancies.

NachoPal avatar Aug 11 '23 13:08 NachoPal