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

add Pendulum assets to Nova wallet #2

Closed
wants to merge 4 commits into from

Conversation

b-yap
Copy link

@b-yap b-yap commented Oct 22, 2024

Closes https://github.com/pendulum-chain/tasks/issues/365

Added the following assets:

Symbol Asset Id ID Scale Precision Existential Deposit Price Id
ASTR XCM(9) 0x0109 18 1000000 astar
vDOT XCM(10) 0x010a 10 1000000 voucher-dot
BNC XCM(11) 0x010b 12 10000000000 bifrost-native-coin
USDC.axl XCM(12) 0x010c 6 1000 axlusdc
EURC.s (Circle EURC) Stellar AlphaNum4 0x020145555243cf4f5a26e2090bb3adcf02c7a9d73dbfe6659cc690461475b86437fa49c71136 12 1000 euro-coin

It looks like USDT has been added already.

I only updated v20 and v21, because both are the only versions with more assets. I only updated chain_dev.json, based on Gianni's previous PR.

@b-yap b-yap requested a review from a team October 22, 2024 07:40
Copy link
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

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

Looks good to me, except for that one icon 👍

"precision": 12,
"type": "orml",
"priceId": "euro-coin",
"icon": "https://raw.githubusercontent.com/novasamatech/nova-utils/master/icons/tokens/white/EUR.svg",
Copy link
Member

Choose a reason for hiding this comment

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

The icon here is for Mykobo's EURC. We should use the icon for the EURC issued by circle.

Copy link
Member

Choose a reason for hiding this comment

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

@prayagd I can see that the linked icon folder only has the EURC.svg for Mykobo. Can we also have the Circle EURC icon as an SVG please?

Copy link
Member

Choose a reason for hiding this comment

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

Pinging again

Copy link

Choose a reason for hiding this comment

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

Sorry somehow missed this message, will share it here

Copy link

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

i already had a EURC circle.svg but nevermind

@ebma
Copy link
Member

ebma commented Nov 8, 2024

I added the other icon and will now create a pull request to the upstream repository.

@ebma ebma closed this Nov 8, 2024
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