Skip to content
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

Hotfix: kusama statemint upgrades #73

Merged
merged 10 commits into from
Apr 5, 2023
Merged

Hotfix: kusama statemint upgrades #73

merged 10 commits into from
Apr 5, 2023

Conversation

bvotteler
Copy link

@bvotteler bvotteler commented Apr 4, 2023

❗ TODO before merging:

  • Apply governance proposal on chopsticks, then
  • Test in chopsticks that sample xcm tx extrinsic (see below) works, sending KSM from Kusama to Kintsugi
  • Rebase from master once Chore: update tests #74 is merged and reenable disabled xcm tests.
  • Wait for governance proposal to be enacted (otherwise it will break automated xcm tests)

Test extrinsic KSM from Kusama to Kintsugi, receiving account id is Alice: 0x630203000100b1200300010100d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d030400000000821a060000000000

Edit: ^ The amount in the extrinsic above was too low. Had to increase it so it was larger than the xcm fees.

@bvotteler bvotteler requested a review from sander2 April 4, 2023 16:11
@bvotteler bvotteler mentioned this pull request Apr 5, 2023
@bvotteler
Copy link
Author

bvotteler commented Apr 5, 2023

Re-ran manual chopsticks tests after adding more dynamic handling to identify V0/V1 vs V2/V3 xcm multi locations.
Tests passed for statemine and kusama transfers.

@sander2 sander2 merged commit ef7fa09 into interlay:master Apr 5, 2023
@bvotteler bvotteler deleted the hotfix-kusama-statemint-upgrades branch April 13, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants