Skip to content

Conversation

@exezbcz
Copy link
Contributor

@exezbcz exezbcz commented Oct 20, 2025

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, invoices must be submitted and payments will be transferred to the Polkadot AssetHub and/or fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#2542

@diogo-w3f diogo-w3f self-assigned this Oct 22, 2025
@diogo-w3f
Copy link
Contributor

@exezbcz thanks for the milestone delivery. I found some problems during the evaluation. Please take a look at the evaluation document and provide the proper fixes. Let me know when I can review it again.

@exezbcz
Copy link
Contributor Author

exezbcz commented Oct 31, 2025

Hi @diogo-w3f, thank you for the thorough evaluation and for spotting these issues.
We've addressed all the identified problems in the following issues:

Regarding the E2E test concern raised in

Our e2e live tests set current chain to CoretimePaseo and HydrationPaseo so the transfer of funds is to those chains see actions

  • from AssetHubPaseo to HydrationPaseo 1s in image
  • from AssetHubPaseoto CoretimePaseo 2n in image

as seen https://assethub-paseo.subscan.io/account/12q5htuBe5CTp4cUqgzZ2NvCBo4Yf8pMPw18cy8aapfz5ay1?tab=xcm_transfer

image

paraspell throws an error if we try from CoretimePaseo to AssetHubPaseo which is NOT what the live e2e are running

image

paraportxyz/sdk#42 (comment)


We believe the codebase is now ready for your review. Please let us know if you need any clarification or have additional feedback.

Thanks again!

@diogo-w3f
Copy link
Contributor

@exezbcz Thanks for the improvements and fixes. I’ve checked everything again, and the only part I couldn’t verify yet is the Hydration interaction. It’s possible the issue is related to funding of the testing account. I’m trying to get funds to test the integration, but I attempted to use the Hydration faucet in Discord and it doesn’t seem to be working.

Could you assist with that? Could the errors I’m seeing during testing be caused by insufficient balance? How did you obtain Hydration testnet tokens to test your application?

Additionally, the images you provided for evaluation are currently unavailable since they are hosted at https://private-user-images.githubusercontent.com/. Could you make them publicly accessible?

@vikiival
Copy link
Contributor

vikiival commented Nov 3, 2025

Hey @diogo-w3f,
feel free to dm me on telegram / X / mail I have hydration testnet tokens so happy to help with this ☺️

@diogo-w3f
Copy link
Contributor

@exezbcz and @vikiival thanks for the answer and for providing funds for running it. However, I still having problems. See the log below. Maybe some configuration or any other asset balance that is missing in my test account. Here is the account I'm using: 15kj2NQ8u8ex9Laz5zou4JCuFkTEzWEfWGPmWstBEkQbcsBq. If needed I can provide its mnemonic in private for you to reproduce the error.

E2E_LIVE=1 E2E_CHAIN=HydrationPaseo E2E_MNEMONIC="<funded mnemonic>" pnpm --filter @paraport/core test:e2e:live

750:757:paraport_sdk/packages/core/e2e/live/xcm.transfer.live.e2e.test.ts
Failed getting Xcm fee ScenarioNotSupported: Scenario ParaToPara not supported for chain CoretimePaseo
    at CoretimePaseo.transferPolkadotXCM (.../@paraspell/sdk-core/dist/index.mjs:8225:15)
    at CoretimePaseo.<anonymous> (.../@paraspell/sdk-core/dist/index.mjs:6601:27)
    at Generator.next (.../@paraspell/sdk-core/dist/index.mjs:332:21)
    at asyncGeneratorStep (.../@paraspell/sdk-core/dist/index.mjs:29:17)
    at _next (.../@paraspell/sdk-core/dist/index.mjs:43:9)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

@exezbcz
Copy link
Contributor Author

exezbcz commented Nov 12, 2025

Hi @diogo-w3f,

Thank you for testing this thoroughly.

The issue you're encountering has been fixed in the latest version. Please fetch the latest changes, install, and rebuild.

Let me know if you encounter any further issues!

@diogo-w3f
Copy link
Contributor

@exezbcz thanks for the information. I've tested again and the teleport to Hydratation worked. The milestone is accepted. Great work!

@diogo-w3f diogo-w3f merged commit 37467d3 into w3f:master Nov 18, 2025
4 of 6 checks passed
@github-actions
Copy link

🪙 Please fill out the invoice form in order to initiate the payment process. Please make sure that you follow the instructions and requirements as laid out in the form as well as our Terms & Conditions. Thank you!

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.

3 participants