feat(code-review): Add PR Review Dashboard frontend [internal] #108534
feat(code-review): Add PR Review Dashboard frontend [internal] #108534vaind wants to merge 2 commits intoivan/pr-review-backendfrom
Conversation
Add event recording, completion webhook handler, cleanup task, and API endpoints for the PR Review Dashboard. This enables tracking and querying Seer code review events.
Add the React UI for the PR Review Dashboard under /explore/pr-review/. Includes PR list with filters, stats chart, and PR detail view.
github-actions
bot
added
the
Scope: Frontend
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| const perPage = cursor?.value ?? 25; | ||
| const start = page * perPage + 1; | ||
| const end = start + prs.length - 1; | ||
| return tct('[start]-[end] of [total]', {start, end, total: totalHits}); |
There was a problem hiding this comment.
Pagination uses cursor value as per-page count
Medium Severity
parseCursor returns value as the first segment of the cursor string (typically a timestamp or ordering boundary), not the page size. Using cursor?.value ?? 25 as perPage will produce wildly incorrect pagination captions on any page after the first. The established codebase pattern (e.g., automationHistoryList.tsx) uses a separate limit variable for per-page size and only reads cursor.offset as the page index.
vaind
changed the title
vaind
force-pushed
the
ivan/pr-review-backend
branch
from
59adf8f to
f972371
Compare
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
1 participant
Summary
Note
This is currently an internal/experimental tool, not meant to be released to customers as is. Take it as a conversation starter and source of internal UX feedback.
/explore/pr-review/pr-review-dashboardfeature flag)PR 3 of 3 for the PR Review Dashboard feature. Based on PR 2 (#108533).
Test plan