Skip to content

Finish merge-fallout cleanup and prep frontend for Vercel deploy - #1258

Merged
GoodnessJohn merged 1 commit into
mainfrom
deploy/vercel-frontend
Aug 14, 2026
Merged

Finish merge-fallout cleanup and prep frontend for Vercel deploy#1258
GoodnessJohn merged 1 commit into
mainfrom
deploy/vercel-frontend

Conversation

@Haroldwonder

Copy link
Copy Markdown
Collaborator

Summary

  • Finishes pending merge-fallout cleanup across backend, frontend, and mobile (component/story/test file consolidation, restored react-i18next dependency)
  • Relaxes the Vite dev-server CSP for HMR/Fast Refresh while keeping the strict production CSP used by the Vercel build unchanged
  • vercel.json (root + frontend/) already configures the static build (vite, frontend/dist output) — verified npm run build succeeds

Verification

  • cd frontend && npm run build succeeds (tsc + vite build, no errors)
  • npx vitest run: 318 passing / 96 failing on this branch vs. 318 passing / 116 failing on main — net improvement, no regressions introduced
  • Remaining test failures are pre-existing and unrelated to this change (stale vi.mocked usage, jsdom scrollIntoView gaps)

Deploy notes

Before deploying on Vercel, set these production env vars in the project settings (see frontend/.env.example):

  • VITE_CONTRACT_ID — deployed SwiftRemit contract id
  • VITE_USDC_ISSUER — USDC issuer account
  • VITE_EURC_ISSUER, VITE_USDC_TOKEN_ID as needed

Test plan

  • Review consolidated component/test file structure
  • Set required VITE_* env vars in Vercel project settings
  • Trigger a Vercel preview deploy from this branch and confirm the app loads

Consolidate component/story/test file layout, restore react-i18next
dependency, and relax dev-server CSP for Vite HMR while keeping the
strict production CSP intact for the Vercel build.
@GoodnessJohn
GoodnessJohn merged commit b3bbdea into main Aug 14, 2026
31 of 85 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.

2 participants