Skip to content

fix: remove user stats from challenge list page#243

Merged
pbrissaud merged 1 commit intomainfrom
fix/remove-user-stats-from-challenges-page
Mar 9, 2026
Merged

fix: remove user stats from challenge list page#243
pbrissaud merged 1 commit intomainfrom
fix/remove-user-stats-from-challenges-page

Conversation

@pbrissaud
Copy link
Member

Summary

  • Removes UserStats component and its ErrorBoundary/Suspense wrapper from the /challenges page
  • Deletes unused components/user-stats.tsx file
  • Stats are already shown on the dashboard — no information is lost

Test plan

  • Visit /challenges — page renders without the 3 stat cards
  • Visit /dashboard — stats still display correctly
  • No TypeScript errors (pnpm typecheck)

Closes #241

🤖 Generated with Claude Code

Remove UserStats component and its ErrorBoundary/Suspense wrapper from
the challenges page. Stats are already shown on the dashboard and were
causing unnecessary complexity with the page's static generation cache.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 9, 2026

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

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Mar 9, 2026 3:57pm

Request Review

@claude
Copy link

claude bot commented Mar 9, 2026

Claude finished @pbrissaud's task in 1m 8s —— View job


PR Review

  • Gather context
  • Review changed files
  • Check for lingering references
  • Post review feedback

This is a clean, focused cleanup PR. Here's what I found:

Changes are correct and complete:

  • components/user-stats.tsx is fully deleted — no orphaned file left behind
  • All imports and JSX usage of UserStats (including its ErrorBoundary/Suspense wrappers) are removed from app/(main)/challenges/page.tsx
  • No lingering references to UserStats or user-stats remain in the main app (the only hits are in the unrelated admin area: app/(admin)/admin/users/)

No issues found:

  • No TypeScript errors introduced (pure deletion of a used component)
  • No dead imports left in challenges/page.tsx
  • The page structure after the removal is clean and logical

Minor observation (non-blocking):
The ChallengesPageContent function still directly queries the DB (getChallenges()) to show the challenge count. This is fine and was pre-existing — just noting it for awareness since it's the only server-side data fetch remaining in that component.

Overall: LGTM. The PR does exactly what it says, nothing more.

@pbrissaud pbrissaud merged commit eac1748 into main Mar 9, 2026
3 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.

Remove statistics on challenge page

1 participant