AstarNetwork / astar-apps

The official Astar Network portal for accessing your tokens and other native features!
https://portal.astar.network/
GNU General Public License v3.0
68 stars 89 forks source link

Add PEN and XLM tokens to Astar portal #1348

Closed b-yap closed 3 weeks ago

b-yap commented 3 months ago

Pull Request Summary

Add support for PEN and XLM asset

This pull request makes the following changes:

Adds

~Test for minimum 0.1 PEN:~ ~https://pendulum.subscan.io//extrinsic/0xfbdd784f2eb845a952a0fefcf5ae76428b42bd4eeffd43936f00e38d768eaafb~ Extrinsic transactions of PEN token in order:

Extrinsic transactions of XLM token in order:

Vercel URL: ~astar-apps-b-yap-bs-projects-fe8df2be.vercel.app~ https://astar-apps-bs-projects-fe8df2be.vercel.app/

bobo-k2 commented 2 months ago

Thank you for the PR. Please send some test tokens to XLoLJBQoMPHMLXYhdFobSpH5GujRoUH8d1sUtaEtoBG7zaS

gtg7784 commented 2 months ago

Hi @b-yap ! Thank you for contribute on our portal, can you send me PEN and XLM for testing it?

This is my Astar wallet address WGeDwTxHVMnbzNS7xkKsxfrabvimPkF96eBa7uYYbTDzNiG

Thank you :)

b-yap commented 2 months ago

@bobo-k2 @gtg7784 Should I send test tokens to both accounts: XLoLJBQoMPHMLXYhdFobSpH5GujRoUH8d1sUtaEtoBG7zaS WGeDwTxHVMnbzNS7xkKsxfrabvimPkF96eBa7uYYbTDzNiG ?

Or any of these 2?

gtg7784 commented 2 months ago

@bobo-k2 @gtg7784 Should I send test tokens to both accounts: XLoLJBQoMPHMLXYhdFobSpH5GujRoUH8d1sUtaEtoBG7zaS WGeDwTxHVMnbzNS7xkKsxfrabvimPkF96eBa7uYYbTDzNiG ?

Or any of these 2?

Please send it to both, We want to test it on each environment.

Thank you :)

b-yap commented 2 months ago

@gtg7784 XLM tokens transferred: https://pendulum.subscan.io/xcm_message/polkadot-a01b30d8091ec0b61088d9ca2550049e55ba3bcc https://pendulum.subscan.io/xcm_message/polkadot-3bb12bfa4fdb443eb5d38f444236875a33a10c84

b-yap commented 2 months ago

@gtg7784 PEN tokens transferred: https://pendulum.subscan.io/xcm_message/polkadot-f1e76d4e05b44b43fedafbfc4e0ab8a51028edd0 https://pendulum.subscan.io/xcm_message/polkadot-b5ac12dd1ed6ef510e948cff74d255df1d36d337

bobo-k2 commented 2 months ago

After selecting Pendulum network

image

XCM component dissapears. This doesn't happen for other networks. I don't see any error in console

image
b-yap commented 2 months ago

@bobo-k2 I have redeployed with a new vercel URL using the updated change: ~https://astar-apps-jnrb64nyx-bs-projects-fe8df2be.vercel.app/~. https://astar-apps-b-yap-bs-projects-fe8df2be.vercel.app/ The problem was the token name. I set it as PEN, but it's actually defined as xcPEN in Astar.

bobo-k2 commented 2 months ago

Hi @b-yap the new Vercel URL is asking me to authenticate, can you fix that?

image
b-yap commented 2 months ago

@bobo-k2 sorry for that. Updated: https://astar-apps-b-yap-bs-projects-fe8df2be.vercel.app/

bobo-k2 commented 2 months ago

When I started testing I had 3 xcPEN on Pendulum. After that I made two transactions from Astar to Pendulum and UI is still showing 3 xcPEN https://astar.subscan.io/extrinsic/0x7c1352ba3daba47a01ff1c7831c7880fd73cc56c8b6e6cce469ae85d61cc088d https://astar.subscan.io/extrinsic/0xba4004fccbf2e0c064f0f9950e287de7f9fe90a177bdbd01062a1537039b6896

image

I transfered 1 XLM.s from Astar to Pendulum and UI still showing 0 XML.s on Pendulum https://astar.subscan.io/extrinsic/0xff7e7c92686805d76377c7b78919951db82e186f561a567f77a86226911fd0b2

image

After reloading the portal shows 0.042 XML.s?!

image
b-yap commented 2 months ago

Moving this to draft, for now.

We identified that the minimum transfer amounts we use so far are not large enough to cover the XCM transfer fees charged on Pendulum. We are currently discussing which values we should use and will update the PR accordingly.

b-yap commented 2 months ago

XLM transfer: Beginning amounts:

Screenshot 2024-07-25 at 4 51 09 PM

Result after transferring 1.01 XLM Pendulum -> Astar

Screenshot 2024-07-25 at 4 56 38 PM

Result after transferring 1.1 XLM Astar -> Pendulum

Screenshot 2024-07-25 at 5 00 38 PM

Result after transferring 1 XLM Pendulum -> Astar

Screenshot 2024-07-25 at 5 07 32 PM
b-yap commented 2 months ago

PEN transfer: Beginning amounts:

Screenshot 2024-07-25 at 11 37 00 PM

Result after transferring 2.01 PEN Pendulum -> Astar:

Screenshot 2024-07-25 at 11 38 36 PM

Result after transferring 2 PEN Pendulum -> Astar:

Screenshot 2024-07-25 at 11 41 22 PM

Result after transferring 2.5 PEN Astar -> Pendulum:

Screenshot 2024-07-25 at 11 43 06 PM
b-yap commented 2 months ago

@bobo-k2 We have increased the minimum transferred amounts, thus gave you more XLMs and PENs to play with. Vercel link: https://astar-apps-bs-projects-fe8df2be.vercel.app/

bobo-k2 commented 1 month ago

Hi @b-yap and thank you for the update. Another problem arise in the meantime. I believe this happens because outdated polkadot.js version. Please merge main into your branch. We uplifted polkadot.js to the latest version recently.

image
b-yap commented 1 month ago

Hello @bobo-k2 , updated: https://github.com/AstarNetwork/astar-apps/pull/1348/commits/9b5c335fd15fd34ca7e2142dcfbd41623383736a

Also updated in vercel: https://astar-apps-b-yap-bs-projects-fe8df2be.vercel.app/

bobo-k2 commented 1 month ago

Hi @b-yap now I am having different error. Did you try Vercel deployment?

image
b-yap commented 1 month ago

Hi @bobo-k2 Not sure what happened in Vercel, my apologies.

I redeployed again (doing yarn -> yarn build all over again):

Screenshot 2024-08-13 at 8 49 07 PM

https://astar.subscan.io/extrinsic/0x7a9ec7bb4776cb5baf986931d71e1a52ec626cd22731c2fda7215c91e0cd5649

impelcrypto commented 1 month ago

Hi @b-yap Could you send some tokens to test? Thanks! 5H6DC1YjyD531FiGxPCpWY3WNJrp5kQ35xHm6wibKuFvWzKW

prayagd commented 1 month ago

@annatekl can you please get @impelcrypto the required tokens to test?

annatekl commented 1 month ago

5H6DC1YjyD531FiGxPCpWY3WNJrp5kQ35xHm6wibKuFvWzKW

sent

impelcrypto commented 3 weeks ago

@annatekl Sorry but the amount of the xcPEN you sent is too low to test bridging tokens, could you help send maybe 20xcPEN?

image

annatekl commented 3 weeks ago

5H6DC1YjyD531FiGxPCpWY3WNJrp5kQ35xHm6wibKuFvWzKW

Shall i send 20 PEN to this address?

impelcrypto commented 3 weeks ago

@annatekl yes please

annatekl commented 3 weeks ago

5H6DC1YjyD531FiGxPCpWY3WNJrp5kQ35xHm6wibKuFvWzKW

sent @impelcrypto

annatekl commented 3 weeks ago

When do I expect the assets to be added to the portal view? @impelcrypto

impelcrypto commented 3 weeks ago

@annatekl Most likely the new version will be deployed next week