Skip to content

feat(reviews): per-reviewer vote breakdown and travel yes rate on the leaderboard - #181

Merged
balebbae merged 5 commits into
mainfrom
devin/1790904596-leaderboard-vote-stats
Oct 2, 2026
Merged

balebbae merged 5 commits into
mainfrom
devin/1790904596-leaderboard-vote-stats

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The admin review leaderboard only showed a total completed count per reviewer. It now also shows how each person is deciding.

GetLeaderboard adds more COUNT(ar.id) FILTER (...) aggregates to the existing query, exposed on store.ReviewerStats as raw counts (percentages are computed client-side):

Accepted, Rejected, Waitlisted int   // completed == accepted + rejected + waitlisted
TravelVotes, TravelYes         int   // travel_vote IS NOT NULL / travel_vote = true

Ranking is unchanged (still by completed).

UI: new columns Accepted / Rejected / Waitlisted (share of that reviewer's completed with the raw count beside it, 67% (8), colored to match VoteBadge) and Travel Yes (67% (4/6), or - when they have given no travel recommendations). Swagger regenerated; integration test now votes with mixed values and travel votes to cover each count.

Link to Devin session: https://app.devin.ai/sessions/e747c0ba90254f8abe73db464417cb55
Open in Devin Desktop: https://app.devin.ai/desktop/session/e747c0ba90254f8abe73db464417cb55?variant=devin
Requested by: @balebbae

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot changed the title feat(reviews): show accepted/rejected/waitlisted counts per reviewer on the leaderboard feat(reviews): per-reviewer vote breakdown, travel yes rate, and decision agreement on the leaderboard Oct 2, 2026
@devin-ai-integration devin-ai-integration Bot changed the title feat(reviews): per-reviewer vote breakdown, travel yes rate, and decision agreement on the leaderboard feat(reviews): per-reviewer vote breakdown and travel yes rate on the leaderboard Oct 2, 2026
@balebbae
balebbae merged commit dad8849 into main Oct 2, 2026
4 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.

1 participant