Skip to content

Fix custom chain icon fallback - #2933

Open
andystorrs wants to merge 1 commit into
DefiLlama:mainfrom
andystorrs:fix-gridmind-icon
Open

Fix custom chain icon fallback#2933
andystorrs wants to merge 1 commit into
DefiLlama:mainfrom
andystorrs:fix-gridmind-icon

Conversation

@andystorrs

Copy link
Copy Markdown
Contributor

Summary

  • Adds the GridMind hosted logo URL to the GridMind Testnet registry entry.
  • Centralizes ChainList icon selection in getChainIcon().
  • Keeps existing chainSlug icons for listed chains and falls back to an HTTP chain.icon before /unknown-logo.png for custom chains.

Validation

  • node -e "import('./utils/index.js').then(({getChainIcon}) => { ... })"
  • node --input-type=module -e "import { pathToFileURL } from 'node:url'; await import(pathToFileURL(process.cwd() + '/constants/additionalChainRegistry/chainid-20260532.js'))"
  • Generated out/rpcs.json with node generate-json.js and confirmed GridMind Testnet includes icon: https://www.grid-mind.com/logo.png.

Note: pnpm test has a Windows-local path issue in this checkout because the test harness imports raw C:\... paths as ESM specifiers; the equivalent file URL syntax import passes.

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