Skip to content

fix(profile): hide total video count in profile header#300

Open
rabble wants to merge 2 commits into
mainfrom
fix/hide-total-video-count
Open

fix(profile): hide total video count in profile header#300
rabble wants to merge 2 commits into
mainfrom
fix/hide-total-video-count

Conversation

@rabble
Copy link
Copy Markdown
Member

@rabble rabble commented Apr 29, 2026

Summary

  • Hide the total Videos stat from profile headers for now
  • Reflow the remaining profile stats into a four-column grid
  • Add a regression test covering the hidden video count and remaining visible stats

Test Plan

  • npx vitest run src/components/ProfileHeader.test.tsx
  • npm run test

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 29, 2026

Deploying divine-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4c328a1
Status: ✅  Deploy successful!
Preview URL: https://0c7b50cc.divine-web.pages.dev
Branch Preview URL: https://fix-hide-total-video-count.divine-web.pages.dev

View logs

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

🚀 Preview Deployment

Property Value
Preview URL https://597d5411.divine-web-fm8.pages.dev
Commit 4c328a1
Branch fix/hide-total-video-count

@NotThatKindOfDrLiz NotThatKindOfDrLiz changed the title Hide profile video count fix(profile): hide total video count in profile header Apr 29, 2026
silentius-satoshi pushed a commit to silentius-satoshi/divine-web that referenced this pull request May 10, 2026
…y with Rules of Hooks

The useEffect for auto-navigating to discovery was conditionally called
inside an empty state block, violating React's Rules of Hooks. This caused
"Rendered fewer hooks than expected" error in production.

Moved the effect to component top level and added the empty-state condition
inside the effect body.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
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