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

Adding BICO token on Base #981

Merged
merged 3 commits into from
Nov 9, 2024
Merged

Conversation

tomarsachin2271
Copy link
Contributor

Description
Adding BICO token on Base.

Tests

Additional context

Deployed BICO token on Base and now adding it to superchain token list.

Metadata

@tomarsachin2271 tomarsachin2271 requested review from wbnns and a team as code owners November 7, 2024 15:29
@wbnns wbnns added the base label Nov 7, 2024
@wbnns wbnns self-assigned this Nov 7, 2024
Copy link
Member

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

@tomarsachin2271

Hi there! Thanks for the PR! CI/CD is currently failing due to the following issue:

error: BICO on chain ethereum token 0xf17e65822b568b3903685a7c9f496cf7656cc6c2 has incorrect name. Got Biconomy Token

In order to fix you'll need to add an override to the metadata as described here:
https://github.com/ethereum-optimism/ethereum-optimism.github.io?tab=readme-ov-file#per-token-overrides

If you have any questions or need further assistance please let us know.

@tomarsachin2271
Copy link
Contributor Author

Added the Token name overrides for ethereum

Copy link
Member

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

ACK

@tomarsachin2271 Thanks for the updates and for building on Base 🔵

@mergify mergify bot merged commit 256a282 into ethereum-optimism:master Nov 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants