Skip to content

feat: add coin metadata for multiple EVM coins #635

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: add coin metadata for multiple EVM coins #635

wants to merge 1 commit into from

Conversation

@nayandas190
Copy link

@nayandas190 nayandas190 commented Oct 22, 2025 •
edited
Loading

Ticket: WIN-7642

  • Add coin metadata for multiple EVM based coins:
  • Testnet Only: phrs
  • Mainnet & Testnet: hypeevm, apechain, ctc, ip, og, lineaeth, sonic, seievm, kaia, and irys.

Test withdrawal Transaction hash:
0xc6889c138e0e3fca860d95c5fac1d0c9fd6b328baa396002476d1e47ff6ed23a
0x188d77fd58c54912c3d8a3a1903c81d8309b2be2ad28634adaadbff2529c76cd
0xb4c9d6e250e75b171e1e8c57be156595c52d3318269e8cec9af229cae7ae41b2
0x611a2e8aeb392b5cce12d079371c49026de44e6e1aebef1d003d8d711f40dd8e
0x16c7ce7484b042a5500d967cbe80ebed9e52a67cf49541a34d152cf335498b6f

@nayandas190 nayandas190 requested review from a team as code owners October 22, 2025 06:17
mullapudipruthvik
mullapudipruthvik reviewed Oct 22, 2025
View reviewed changes
Copy link
Contributor

@mullapudipruthvik mullapudipruthvik left a comment

Choose a reason for hiding this comment

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

are there not tests to verify this ?
if not add a recording to validate your changes

prithvishet2503
prithvishet2503 requested changes Oct 22, 2025
View reviewed changes
Copy link
Contributor

@prithvishet2503 prithvishet2503 left a comment

Choose a reason for hiding this comment

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

also add the ApiKeyProvider, minGasLimit , defaultGasLimit ,defaultGasLimitNum and the currency icons

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image image image image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image image image image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

image image image image

@nayandas190
Copy link
Author

nayandas190 commented Oct 26, 2025

also add the ApiKeyProvider, minGasLimit , defaultGasLimit ,defaultGasLimitNum and the currency icons

Currency icons are already present in master.

@nayandas190 nayandas190 marked this pull request as draft October 27, 2025 05:07
OttoAllmendinger
OttoAllmendinger reviewed Oct 31, 2025
View reviewed changes
Copy link
Contributor

@OttoAllmendinger OttoAllmendinger left a comment

Choose a reason for hiding this comment

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

Automatic comment to flush this PR from my review queue. 🚽🚽🚽
Please re-request another review from the respective team(s) when you are ready for reviews.

@nayandas190 nayandas190 marked this pull request as ready for review November 3, 2025 11:26
@prithvishet2503
Copy link
Contributor

prithvishet2503 commented Nov 5, 2025

0xc6889c138e0e3fca860d95c5fac1d0c9fd6b328baa396002476d1e47ff6ed23a
0x188d77fd58c54912c3d8a3a1903c81d8309b2be2ad28634adaadbff2529c76cd
0xb4c9d6e250e75b171e1e8c57be156595c52d3318269e8cec9af229cae7ae41b2
0x611a2e8aeb392b5cce12d079371c49026de44e6e1aebef1d003d8d711f40dd8e
0x16c7ce7484b042a5500d967cbe80ebed9e52a67cf49541a34d152cf335498b6f

pls add the url link to verify

@nayandas190
Copy link
Author

nayandas190 commented Nov 5, 2025

tip: https://aeneid.storyscan.io/tx/0xb4c9d6e250e75b171e1e8c57be156595c52d3318269e8cec9af229cae7ae41b2
tflow: https://evm-testnet.flowscan.io/tx/0x611a2e8aeb392b5cce12d079371c49026de44e6e1aebef1d003d8d711f40dd8e
tlineaeth: https://sepolia.lineascan.build/tx/0xc6889c138e0e3fca860d95c5fac1d0c9fd6b328baa396002476d1e47ff6ed23a
tplume: https://testnet-explorer.plume.org/tx/0x188d77fd58c54912c3d8a3a1903c81d8309b2be2ad28634adaadbff2529c76cd
tapechain: https://curtis.apescan.io/tx/0x785c294c0fc55de49244c1f5ca91fac6276b56611a8921a75222bbefa0182893

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@OttoAllmendinger OttoAllmendinger OttoAllmendinger left review comments

@mullapudipruthvik mullapudipruthvik mullapudipruthvik left review comments

@prithvishet2503 prithvishet2503 Awaiting requested review from prithvishet2503

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@nayandas190 @prithvishet2503 @OttoAllmendinger @mullapudipruthvik