Skip to content

fix: add Base to default chains so its icon renders out of the box#508

Open
tenderdeve wants to merge 1 commit into
family:mainfrom
tenderdeve:fix/base-chain-icon
Open

fix: add Base to default chains so its icon renders out of the box#508
tenderdeve wants to merge 1 commit into
family:mainfrom
tenderdeve:fix/base-chain-icon

Conversation

@tenderdeve
Copy link
Copy Markdown

Summary

  • Base (chain ID 8453) had its icon SVG added in add: zora + base logos #418 but was not included in the default chains array
  • The chain icon mapping was never triggered because Base wasn't in the list
  • Add base to the default chains in defaultConfig.ts and defaultTransports.ts

Files:

  • packages/connectkit/src/defaultConfig.ts
  • packages/connectkit/src/defaultTransports.ts

Fixes #438

Test plan

  • Base chain icon renders in the network selector without custom config
  • Other chain icons (Arbitrum, Mainnet, Polygon, Optimism) still render
  • Default config connects to Base correctly

Base (chain ID 8453) had its icon SVG added in family#418 but was not
included in the default chains array, so the chain icon mapping
was never triggered. Add Base to the default chains in
defaultConfig.ts and defaultTransports.ts.

Fixes family#438
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 28, 2026

@tenderdeve is attempting to deploy a commit to the Aave Team on Vercel.

A member of the Team first needs to authorize it.

@tenderdeve tenderdeve marked this pull request as ready for review May 11, 2026 09:22
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.

[BUG] Base network icon logo seems to not be working out of the box

1 participant