-
Notifications
You must be signed in to change notification settings - Fork 835
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
[asset-conversion] Add runtime api to polkadot-js/api #108
Comments
Needs polkadot-js/api#5700 |
Can it be closed? both PRs are already merged, not sure if there is still something missing or not. |
iirc, initially, we've added some stuff to the polkadot.js, but I am not sure if it is still compatible, because there were several refactors in asset-conversion pallet, so we should check this. @franciscoaguirre Cisco has already fixed some stuff within XCMv4, I would say we need to revisit that #1129 again, what would be the best option |
I pinned version 3 so we can do a storage migration whenever |
yes, one thing is storage and another thing is public APIs for indexers/wallets and so on, maybe for them it would be better to use Another thing, I think we should maybe add some |
Shouldn't this issue be in PJS apps? |
Yeah, it should be. |
polkadot-js/api#5694 but also looks like it was already done. |
E.g.: FungiblesApi: https://github.com/polkadot-js/api/blob/3f90f72bd69c49a7ef6902366572c90c1efcdb52/packages/types/src/interfaces/fungibles/runtime.ts
The text was updated successfully, but these errors were encountered: