Skip to content

Fix JSON formatting in FuraCoin metadata - #1890

Open
Furacoin168 wants to merge 5 commits into
MetaMask:masterfrom
Furacoin168:master
Open

Fix JSON formatting in FuraCoin metadata#1890
Furacoin168 wants to merge 5 commits into
MetaMask:masterfrom
Furacoin168:master

Conversation

@Furacoin168

@Furacoin168 Furacoin168 commented Aug 1, 2026

Copy link
Copy Markdown

Add token metadata


Note

Low Risk
Static token metadata only; no runtime or security-sensitive code changes.

Overview
Adds FuraCoin (FURA, 18 decimals) to the Ethereum mainnet ERC-20 catalog at contract 0xe1631864495a2251Ae092d678111ACE32eE28aD4, using the same metadata shape as other tokens (name, symbol, decimals, erc20, logo path under ./icons/...).

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

@Furacoin168
Furacoin168 requested review from a team and MRabenda as code owners August 1, 2026 02:14

@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

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6f6a1ea. Configure here.


"erc20": true,

"logo": "./icons/eip155:1/erc20:0xe1631864495a2251Ae092d678111ACE32eE28aD4.png"}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing referenced token logo

High Severity

The logo field points to ./icons/eip155:1/erc20:0xe1631864495a2251Ae092d678111ACE32eE28aD4.png, but that icon file is not present in the repository. Token metadata is incomplete without the image, so the logo cannot render and asset verification/test/caip.js logo existence checks fail.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6f6a1ea. Configure here.

@Furacoin168

Copy link
Copy Markdown
Author

He añadido el logo en icons/eip155:1/erc20/0xe1631864495a2251Ae092d678111ACE32eE28aD4.png para completar la metadata y resolver la advertencia del bot. Por favor, re-ejecuten los checks y revisen cuando puedan. Contacto: roselemerald@gmail.com / investment@emeraldstrade.com

Comentario corto en inglés (si prefieres)Added the logo at icons/eip155:1/erc20/0xe1631864495a2251Ae092d678111ACE32eE28aD4.png to complete the metadata and fix the Cursor bot warning. Please re-run checks and review when possible. Contacts: roselemerald@gmail.com / investment@emeraldstrade.com

… JSON

- Place logo at icons/eip155:1/erc20:0xe163...D4.png (matches metadata logo field)
- Remove misplaced root-level PNG from earlier upload
- Normalize JSON formatting (no blank lines, proper closing brace)
@arealicehole

Copy link
Copy Markdown

Fix applied

Resolved the Missing referenced token logo finding:

  1. Added logo at the path referenced by metadata:
    icons/eip155:1/erc20:0xe1631864495a2251Ae092d678111ACE32eE28aD4.png
    • 256×256 PNG, RGBA, transparent background
  2. Removed misplaced root-level duplicate:
    erc20:0xe1631864495a2251Ae092d678111ACE32eE28aD4.png
  3. Tidied metadata JSON formatting (logo path unchanged and correct)

Please re-run checks / re-review.

Contacts: roselemerald@gmail.com / investment@emeraldstrade.com

@arealicehole

Copy link
Copy Markdown

Path note (please ignore any guide saying otherwise)

The correct logo path uses a colon in the filename, not a /erc20/ subdirectory:

✅ Correct (what is in this PR now):
icons/eip155:1/erc20:0xe1631864495a2251Ae092d678111ACE32eE28aD4.png

❌ Incorrect (do not upload here):
icons/eip155:1/erc20/0xe1631864495a2251Ae092d678111ACE32eE28aD4.png

Metadata logo field matches the correct colon path. No further file moves needed on our side.

@Furacoin168

Copy link
Copy Markdown
Author

He añadido el logo en la ruta referenciada y se han eliminado duplicados; la imagen es 256×256 PNG (RGBA). ¿Podrían re-ejecutar los checks y revisar/aprobar cuando puedan? Gracias.

@Furacoin168

Copy link
Copy Markdown
Author

ES:@MRabenda urgente por favor — necesitamos que se aprueben los workflows en PR #1890 para que los checks se ejecuten. El logo ya está en icons/eip155:1/erc20/0xe1631864…png y tenemos ticket de soporte #120870179 solicitando revisión manual/escalado. ¿Podrías ayudar a aprobar y revisar pronto? Gracias.

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.

2 participants