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

CropBytes CBX Token #917

Merged
merged 2 commits into from
Aug 29, 2024
Merged

CropBytes CBX Token #917

merged 2 commits into from
Aug 29, 2024

Conversation

kevcb
Copy link

@kevcb kevcb commented Aug 28, 2024

  • CBX Folder created in data
  • data.json added for CBX Token (eth, base)
  • logo.svg added for CBX token

- CBX Folder created in data
- data.json added for CBX Token
- logo.svg added for CBX token
@kevcb kevcb requested review from wbnns and a team as code owners August 28, 2024 07:22
@wbnns wbnns added the base label Aug 28, 2024
@wbnns wbnns self-assigned this Aug 28, 2024
Copy link
Collaborator

@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.

@kevcb

Thanks for the PR! CI is failing and this cannot be merged due to the following issues:

error: CBX on chain ethereum token 0x37fc4b48ce93469dbea9918468993c735049642a has incorrect name. Got CropBytes
error: CBX on chain base token 0xD988824d7fA1998D0ABBab2F79A82aF762886caC has incorrect name. Got CropBytes

The name should be CropBytes rather than CropBytes Token in the metadata. Alternatively, you can override the name in the metadata as described here:
https://github.com/ethereum-optimism/ethereum-optimism.github.io?tab=readme-ov-file#per-token-overrides

Hope that helps! Please let us know once updated and we'll rereview.

 "name" in json file updated to "CropBytes"
@kevcb
Copy link
Author

kevcb commented Aug 29, 2024

@wbnns

I will be updating the same and pushing it.

Thanks

Copy link
Collaborator

@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

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

Copy link
Contributor

mergify bot commented Aug 29, 2024

Merge failed. Please see automated check logs for more details.

@nitaliano nitaliano merged commit fb8eecc into ethereum-optimism:master Aug 29, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants