Skip to content

Create erc20:0x2970134F4e01a8AE77b36c37c7b688b4ADEC76CF.svg - #1919

Open
armaverseio wants to merge 1 commit into
MetaMask:masterfrom
armaverseio:patch-5
Open

Create erc20:0x2970134F4e01a8AE77b36c37c7b688b4ADEC76CF.svg#1919
armaverseio wants to merge 1 commit into
MetaMask:masterfrom
armaverseio:patch-5

Conversation

@armaverseio

@armaverseio armaverseio commented Aug 21, 2026

Copy link
Copy Markdown

Note

Low Risk
Asset-only change: a single static SVG icon with no logic, auth, or data handling.

Overview
Adds a new ERC-20 token icon for address 0x2970134F4e01a8AE77b36c37c7b688b4ADEC76CF on BNB Chain (eip155:56).

Reviewed by Cursor Bugbot for commit efc13ea. Bugbot is set up for automated code reviews on this repo. Configure here.

@armaverseio
armaverseio requested review from a team and MRabenda as code owners August 21, 2026 07:26

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit efc13ea. Configure here.

<rect width="1080" height="1080" fill="#000000"/>
<path fill="#ffffff" d="M540 198 L228 842 L342 842 L540 437 L632 627 L748 627 Z"/>
<path fill="#ffffff" d="M686 736 L737 842 L852 842 L800 736 Z"/>
</svg>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orphan icon missing metadata

Medium Severity

This icon has no matching metadata/eip155:56/erc20:0x2970134F4e01a8AE77b36c37c7b688b4ADEC76CF.json entry. Asset lookup and token-list generation only read metadata, so the file is unused and the token logo will never surface to clients.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit efc13ea. Configure here.

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.

1 participant