CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix(web): remove orphaned `error`/`result` references in settings page #4094

Merged⚡ AI-generatedXLccantynz wants to mergeclaude/fix-site-display-t3HP0claude/build-email-service-3R7eoopened Apr 25, 2026
ccantynzcommented Apr 25, 2026

Originally written by @ccantynz-alt on GitHub.
Imported from https://github.com/ccantynz-alt/AlecRae.com/pull/37


Build was failing with Cannot find name 'error' in two places in apps/web/app/(dashboard)/settings/page.tsx — leftovers from a refactor:

  • ProfileSection had a duplicate {error && ...} block alongside the correct {saveError && ...} block; removed the dead duplicate.
  • DangerZone referenced undefined error and result variables in dangling JSX; removed.

Production deploy was broken because next build exited 1 on type checking. Local build now: 29/29 static pages, zero errors.

https://claude.ai/code/session_019uxxTpXVK4rw9XzatpX2hs

ccantynzcommented Apr 25, 2026

Originally written by @vercel[bot] on GitHub.


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

Project Deployment Actions Updated (UTC)
48co.ai Error Error Apr 25, 2026 11:45am
alec-rae-com-admin Ready Ready Preview, Comment Apr 25, 2026 11:45am
alecrae Ready Ready Preview, Comment Apr 25, 2026 11:45am

Request Review

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into claude/build-email-service-3R7eo.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts