Skip to content

Fix swapped JSX closing tags in not-found.tsx#47

Merged
naheel0 merged 2 commits intonot-found-pagefrom
copilot/sub-pr-42
Feb 21, 2026
Merged

Fix swapped JSX closing tags in not-found.tsx#47
naheel0 merged 2 commits intonot-found-pagefrom
copilot/sub-pr-42

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

</Button> and </Link> were swapped in the CTA section of the 404 page, producing an invalid JSX tree and failing the CI lint step with JSX element 'Button' has no corresponding closing tag.

Fix

  • Corrected tag order so </Button> closes before </Link>:
// Before (invalid)
<Link href="/">
  <Button variant="primary" ...>
    <Home size={20} />
    Back to Safety
  </Link>   {/* ← wrong: closes Link before Button */}
</Button>

// After
<Link href="/">
  <Button variant="primary" ...>
    <Home size={20} />
    Back to Safety
  </Button>
</Link>

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI mentioned this pull request Feb 21, 2026
10 tasks
@vercel
Copy link
Contributor

vercel bot commented Feb 21, 2026

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

Project Deployment Actions Updated (UTC)
readme-gen-ai Ready Ready Preview, Comment Feb 21, 2026 4:59pm

…-found.tsx

Co-authored-by: naheel0 <191262736+naheel0@users.noreply.github.com>
Copilot AI changed the title [WIP] Add error page Fix swapped JSX closing tags in not-found.tsx Feb 21, 2026
Copilot AI requested a review from naheel0 February 21, 2026 17:00
@naheel0 naheel0 marked this pull request as ready for review February 21, 2026 17:04
@naheel0 naheel0 requested a review from adithyanmkd as a code owner February 21, 2026 17:04
@naheel0 naheel0 merged commit 676657c into not-found-page Feb 21, 2026
3 checks passed
@naheel0 naheel0 deleted the copilot/sub-pr-42 branch February 21, 2026 17:06
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