Skip to content

fix(mobile): contest followers screen blank when limit exceeds 100#14337

Merged
dylanjeffers merged 1 commit into
mainfrom
dylan/fix-mobile-contest-followers-blank
May 15, 2026
Merged

fix(mobile): contest followers screen blank when limit exceeds 100#14337
dylanjeffers merged 1 commit into
mainfrom
dylan/fix-mobile-contest-followers-blank

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

The mobile Contest Followers screen (the dedicated list opened from the
"FOLLOWERS (N)" card on a remix contest page) was rendering as a blank
white screen under its "FOLLOWERS" header — no rows, no skeletons, no empty
state.

Root cause: the screen requested useEventFollowers({ eventId, limit: 200 }),
while the equivalent web caller (ContestFollowersModal) uses limit: 100.
Above that ceiling the /events/{eventId}/followers response comes back
empty, so userIds resolved to [], isPending flipped to false, and the
shared UserList rendered nothing.

The avatar pile on the contest details tab kept working because it requests
limit: 7 via EventFollowersCard, well under the ceiling.

Fix

  • ContestFollowersScreen.tsx now requests limit: 100, matching the web modal.
  • Updated the surrounding docstring to record why we hold at 100 so future
    readers don't bump it back up.

Test plan

  • Open a remix contest with a few followers on mobile
  • Tap the "FOLLOWERS (N)" card on the details tab
  • Confirm the dedicated screen now lists the follower rows instead of
    rendering a blank "FOLLOWERS" header
  • Confirm the avatar pile on the details tab still renders (limit 7,
    unchanged path)
  • Web: confirm the ContestFollowersModal is unaffected (no changes
    touched the hook or the web caller)

🤖 Generated with Claude Code

The mobile ContestFollowersScreen requested `limit: 200` for the
follower list, while the equivalent web modal (`ContestFollowersModal`)
uses `limit: 100`. Requesting above that ceiling comes back empty, so
the UserList rendered no rows — only the "FOLLOWERS" header.

Match the web caller (`limit: 100`) so the screen actually populates
when the user taps the followers card. Updates the surrounding comment
to record the constraint so future readers do not raise it back.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 15, 2026

⚠️ No Changeset found

Latest commit: ba7a069

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dylanjeffers dylanjeffers merged commit 3f4eac3 into main May 15, 2026
3 checks passed
@dylanjeffers dylanjeffers deleted the dylan/fix-mobile-contest-followers-blank branch May 15, 2026 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant