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

added reserve asset transfer script #6990

Open
wants to merge 3 commits into
base: papi_chopsticks_docs
Choose a base branch
from
Open

Conversation

x3c41a
Copy link
Contributor

@x3c41a x3c41a commented Dec 23, 2024

Added reserve asset transfer script.

To run chopsticks use:
bunx @acala-network/chopsticks@latest xcm -r configs/westend-override.yaml -p configs/westend-asset-hub-override.yaml -p configs/westend-penpal-override.yaml

fun: XcmV3MultiassetFungibility.Fungible(1_000_000_000_000n),
}
}),
XcmV4Instruction.TransferReserveAsset({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you're using V5, you should use InitiateTransfer

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah penpal is still on v4, we should fix that

},
);
const r = await ahToWnd.signAndSubmit(aliceSigner);
expect(r).toBeTruthy();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this assertion enough?


const ahToWnd = AHApi.tx.PolkadotXcm.execute({
message: msg,
max_weight: { ref_time: 81834380000n, proof_size: 823193n },
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should get it from the runtime API

@@ -387,3 +387,99 @@ test("Initiate Teleport with remote fees", async () => {
const r = await ahToWnd.signAndSubmit(aliceSigner);
expect(r).toBeTruthy();
})

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't there be a package.json somewhere?

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.

2 participants