Add GOOGL, INTC, AAPL, MSFT, LLY, and PTY Base tokens - #245
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughThe exported ChangesToken registry
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Line 1 in 59f9722
Replace the cropped PTY logo
When the Base registry returns /images/PTY.png, every UI rendering that logo displays a visibly truncated PIMCO wordmark because the final “O” is cut off at the image's right edge; circular clipping in the token components makes the defect even more apparent. Replace this file with an uncropped source or a clean issuer logo before publishing the PTY asset.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Of the tokens launched on Base 2026-07-27, GOOGL.png, META.png, AAPL.png and MSFT.png were already present and tracked, so LLY is the only new asset. Sourced from the Parqet CDN at 1000x1000, matching the existing logos. Force-added past the `*.png` gitignore, as with the existing assets. PTY is not being launched and is deliberately excluded. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
fd5e50f to
b8e9032
Compare
alastairong1
changed the title
PTY is a PIMCO closed-end fund, so the asset is PIMCO's own wordmark, following the issuer-logo convention this repo already uses for funds (wtQQQM -> invesco.png, wtVWO -> vanguard.png, wtDRAM -> roundhill.png). Rendered from the Wikimedia Commons PIMCO vector at 1000x1000: wordmark knocked out in white on the brand navy (#25477f), matching the LLY style. The Parqet CDN was not usable here — its native PIMCO asset is 100x100, so its 1000px render was a 10x upscale with the trailing "O" cropped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
alastairong1
changed the title
INTC replaces META in the 2026-07-27 launch set. META.png already exists in the repo and is left untouched. Sourced from the Parqet CDN at 1000x1000; the native asset there is an SVG, so this is a true vector render rather than an upscale. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
alastairong1
changed the title
findolor
changed the title
3 participants
Summary
Adds the six Base tokens launched on 2026-07-27 to the website token configuration and supplies the three logo assets that were not already present.
0x6a2357Df4975C667B171bE53dA6FFe6deBf7030cEquity.US.GOOGL/USD0xf567652fC2d7Db8D5469fD06AEbe8C9c4372d722Equity.US.INTC/USD0x1020EC8Aa3f709a1f3D8705cdBa89b950451bd88Equity.US.AAPL/USD0x515A3Ac2a6aB590bDFa970caFFFd7fAdC680886EEquity.US.MSFT/USD0x892CcF5E75f4a7Ee6402971a1587F65BEE4d52bdEquity.US.LLY/USD0xb6D779A79E7493ed821a65D52bA419F0F6D5dD0aThe wrapper, unwrapped SFT, and receipt mappings were refreshed from the production
sft-base/1.0.12subgraph. Pyth feed IDs were refreshed from the official catalog.Logos
INTC.png,LLY.png, andPTY.pngas 1000×1000 assets.GOOGL.png,AAPL.png, andMSFT.png.Related registry change
Validation
bun run lintpassed.tests/lib/network.test.ts: 37 passedtests/lib/queries/priceFeeds.test.ts: 4 passedsvelte-check: 0 errors, 0 warningsmain.200 image/png.