393 use fee per second from asset metadata in xcm config

ebma commented 3 months ago

This PR adds changes that were reverted before. The original PR is #410.

We only want to merge this after the dependency upgrade to v0.9.42 is merged and released.

TODO

[x] We still need to change the conversion function for MultiLocation <> CurrencyId. Currently, we are still using the hard-coded logic that manually maps these two but we also want to use the asset registry for this so no further runtime upgrades are needed for adding assets. Thus, we should change the conversion function similar to what is done here.

Closes #393.

gianfra-t commented 3 months ago

Last commit updated and simplified the implementation now. I will leave a brief description since the conversation is now too long.

We now don't modify at all the location before trying to convert nor we convert while getting the fee per second. This is all possible because now we use the local PoV for our assets in the asset registry (example in the mock). xTokens::tansfer_multiasset now works with the local PoV (Thanks @TorstenStueber for figuring that out)

TorstenStueber commented 3 months ago

The Foucoco SelfLocation is still (parents: 1, Parachain(...)), can you change that please?

ebma commented 3 months ago

Let's merge and roll it out on Foucoco today?

pendulum-chain / pendulum

393 use fee per second from asset metadata in xcm config #432

TODO