Skip to content

feat: add WABTC icon mapping - #160

Merged
claudiiafg merged 1 commit into
mainfrom
feat/add-wabtc-icon-mapping
Nov 3, 2026
Merged

claudiiafg merged 1 commit into
mainfrom
feat/add-wabtc-icon-mapping

Conversation

@semeano

@semeano semeano commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Add tWABTC icon (Wrapped Alpen BTC) — alpen_testnet/erc20/wrapped_alpen_btc_0xa2903243f20eede1839cb07ee5ce6b2ba5820000

Adds

  • WABTC icon mapping | Wrapped Alpen BTC on Alpen Testnet has no icon, so Ledger Wallet shows the letter fallback. LIVE-37869

Change

  • Reuses Alpen artwork | The token id joins the ALPEN_Testnet entry, so it resolves to the existing ALPEN_Testnet.png; no new image.

Risks

  • Affects | Ledger Wallet desktop and mobile → @ledgerhq/crypto-icons index.
  • Same mark as network | The token shows the Alpen Testnet "A"; no WABTC artwork exists. Mainnet WABTC is out of scope.

Review roadmap

  1. assets/_record.json | start here: one id added to ALPEN_Testnet, copied verbatim from CAL.
  2. Skim | assets/index.json, generated by pnpm generate:index.

Tested

  • CI gates locally | build, knip, lint, test, check:mapping, index diff and verify-icons.js pass; the id resolves on CAL next.
  • Still manual | Storybook not run; the PNG is already on the CDN, so the icon should render before merge.

Related

Checklist

  • 144×144 PNG (existing ALPEN_Testnet.png, unchanged)
  • Compressed with zopflipng (≤ 50 KB) (existing file, unchanged)
  • _record.json updated (alphabetical order)
  • index.json regenerated

@semeano
semeano requested review from a team and a balanced review from Copilot November 3, 2026 14:07
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying crypto-icons-storybook-react-native with  Cloudflare Pages  Cloudflare Pages

Latest commit: 60ef6e7
Status: ✅  Deploy successful!
Preview URL: https://0f90c324.crypto-icons-storybook-react-native.pages.dev
Branch Preview URL: https://feat-add-wabtc-icon-mapping.crypto-icons-storybook-react-native.pages.dev

View logs

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are recorded for the tWABTC icon mapping update.

No open issues 🎉

What changed in this PR

Adds an icon mapping for Wrapped Alpen BTC (tWABTC) on Alpen Testnet so Ledger Wallet can display the existing Alpen Testnet artwork instead of a letter fallback.

Changes:

  • Registers the token ID under ALPEN_Testnet, reusing the artwork added in #157.
  • Updates the generated index to resolve the token to ALPEN_Testnet.png; no image changes.
File Description
assets/​index.json Maps the tWABTC token ID to the existing Alpen Testnet PNG.
assets/​_record.json Adds the tWABTC token ID to the ALPEN_Testnet entry.

Review effort: Balanced

@claudiiafg
claudiiafg merged commit cf63706 into main Nov 3, 2026
13 checks passed
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.

3 participants