Skip to content

feat: migrate image tools to Next.js#65

Draft
rossgalloway wants to merge 2 commits into
mainfrom
feat/nextJS
Draft

feat: migrate image tools to Next.js#65
rossgalloway wants to merge 2 commits into
mainfrom
feat/nextJS

Conversation

@rossgalloway

@rossgalloway rossgalloway commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • migrate the image upload tool from Vite and Vercel Functions to Next.js 16 App Router route handlers
  • preserve token and chain uploads, GitHub OAuth preview returns, repository targeting, and automatic chain path creation
  • harden chain ID validation and OAuth callback state handling
  • update the header with the Yearn tokenAssets GitHub link and revised sign-in styling

Why

This change keeps the framework (nextJS) consistent with the yearn.fi site, which recently switched from Vite to nextJS.

Impact and risk

  • This is a full overhaul of the website framework so we should confirm all is working correctly.
  • image-tools now runs with next dev, next build, and next start

Validation

  • bun test — 25 passing
  • bun run typecheck
  • bun run build
  • conflict-free merge tree against current origin/main

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
token-assets Ready Ready Preview, Comment Jul 16, 2026 4:02pm
token-assets-ui Error Error Jul 16, 2026 4:02pm

Request Review

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