Skip to content

Add token logo downloads#136

Open
rossgalloway wants to merge 3 commits into
mainfrom
feat/add-token-logos
Open

Add token logo downloads#136
rossgalloway wants to merge 3 commits into
mainfrom
feat/add-token-logos

Conversation

@rossgalloway

Copy link
Copy Markdown
Collaborator

Summary

  • Add token-assets URL helpers for SVG and PNG token logo variants
  • Show available token logo downloads on token detail pages
  • Add direct blob-backed downloads for individual logo files and all files together
  • Link token pages to token-assets.yearn.fi for adding or updating logos

Validation

  • bunx biome check lib/assets.ts src/routes/Collection.tsx
  • VITE_CDN_URL=/api/cdn/ bun run build

Note: full bun run lint still reports pre-existing lint issues outside this change.

@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cms Ready Ready Preview, Comment Jul 7, 2026 10:25pm

Request Review

@rossgalloway

Copy link
Copy Markdown
Collaborator Author

Pushed a Vercel build fix in ae23a5d.

The preview install was resolving motion-dom to the latest version from framer-motion's ^12.23.12 transitive range. framer-motion@12.23.12 imports activeAnimations, which is exported by motion-dom@12.23.12 but not by the newer resolved package. I pinned motion-dom@12.23.12 as an explicit app dependency so package-root installs resolve the compatible version.

Validation:

  • bunx biome check package.json lib/assets.ts src/routes/Collection.tsx
  • VITE_CDN_URL=/api/cdn/ bun run build
  • temp packages/app install/build smoke test confirmed motion-dom@12.23.12

@rossgalloway rossgalloway marked this pull request as ready for review July 7, 2026 22:37
@rossgalloway rossgalloway requested a review from w84april July 7, 2026 22:37
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