Skip to content

feat(webapp): add squad top members links#5831

Merged
idoshamun merged 8 commits intomainfrom
codex/squad-top-members-pr
Apr 1, 2026
Merged

feat(webapp): add squad top members links#5831
idoshamun merged 8 commits intomainfrom
codex/squad-top-members-pr

Conversation

@idoshamun
Copy link
Copy Markdown
Member

@idoshamun idoshamun commented Apr 1, 2026

Summary

  • add a Top members section to squad pages and hydrate it with the squad fetch
  • add overflow modal support and server-rendered SEO links for privileged members and top members
  • update the squad loading skeleton and squad page tests for the new header content

Verification

  • node ./scripts/typecheck-strict-changed.js
  • pnpm --filter webapp exec eslint 'pages/squads/[handle]/index.tsx' 'tests/SquadFeedPage.tsx'
  • pnpm --filter shared exec eslint 'src/graphql/sources.ts' 'src/graphql/squads.ts' 'src/components/squads/Members/PrivilegedMemberItem.tsx' 'src/components/squads/SquadPageHeader.tsx' 'src/components/modals/common.tsx' 'src/components/modals/common/types.ts' 'src/components/modals/squads/SquadUsersModal.tsx' 'src/components/modals/squads/PrivilegedMembersModal.tsx' 'src/components/modals/squads/TopMembersModal.tsx'
  • pnpm --filter shared exec eslint 'src/components/errors/SquadLoading.tsx'

Notes

  • local Jest file runs are still blocked in this environment by the existing jest-environment-jsdom setup failure before test execution

Preview domain

https://codex-squad-top-members-pr.preview.app.daily.dev

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Apr 1, 2026 8:40am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Apr 1, 2026 8:40am

Request Review

@idoshamun idoshamun merged commit ce2d215 into main Apr 1, 2026
12 checks passed
@idoshamun idoshamun deleted the codex/squad-top-members-pr branch April 1, 2026 08:40
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