fix: SSR vote counts for session detail ticks and add comment count button#875
fix: SSR vote counts for session detail ticks and add comment count button#875marcodejongh merged 2 commits intomainfrom
Conversation
…utton
Tick-level VoteButtons were hardcoded with initialUpvotes={0} and
initialUserVote={0}, which prevented them from ever loading real vote
counts. This adds upvotes to the SessionDetailTick type and batch-fetches
tick vote counts in the backend resolver so they render correctly on
initial page load. Also replaces standalone CommentSection with
FeedCommentButton (with new defaultExpanded prop) to show the SSR'd
comment count.
https://claude.ai/code/session_015CCDzunZzCLDf2vkf4yvVW
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Claude Review✅ Ready to merge - Minor test coverage gap noted below, but nothing blocking. Issues
|
The VoteButton mock now captures initialUpvotes, and the tick VoteButton test asserts it equals 3 (the value from the fixture). This directly validates the fix that was the purpose of the previous commit. https://claude.ai/code/session_015CCDzunZzCLDf2vkf4yvVW
Claude Review✅ Ready to merge - No significant issues found. The implementation correctly adds SSR vote counts by:
Tests are appropriately updated to cover the new functionality. |
Tick-level VoteButtons were hardcoded with initialUpvotes={0} and
initialUserVote={0}, which prevented them from ever loading real vote
counts. This adds upvotes to the SessionDetailTick type and batch-fetches
tick vote counts in the backend resolver so they render correctly on
initial page load. Also replaces standalone CommentSection with
FeedCommentButton (with new defaultExpanded prop) to show the SSR'd
comment count.
https://claude.ai/code/session_015CCDzunZzCLDf2vkf4yvVW