Skip to content

Feat/watchlist pr datatable#667

Merged
e35ventura merged 8 commits intoentrius:testfrom
plind-junior:feat/watchlist-pr-datatable
Apr 26, 2026
Merged

Feat/watchlist pr datatable#667
e35ventura merged 8 commits intoentrius:testfrom
plind-junior:feat/watchlist-pr-datatable

Conversation

@plind-junior
Copy link
Copy Markdown
Contributor

@plind-junior plind-junior commented Apr 21, 2026 •
edited
Loading

Summary

  • Watchlist PR DataTable: Replace simple row layout in the Pull Requests tab with the reusable DataTable component — columns for PR number, title, repository, author (with avatar), status chip, score, and watchlist star
  • Clear dialog styling: Style the "Clear all pinned items?" confirmation dialog to match the project dark theme (JetBrains Mono font, background.default, themed borders and buttons)
  • Sidebar hover fix: Extend leaderboard row hover highlight to cover full width in LeaderboardSidebar
  • MinersList props fix: Update WatchlistButton usage from old githubId prop to new category + itemKey API

Test plan

  • Navigate to Watchlist page, star some PRs, verify they render in a proper table under the Pull Requests tab
  • Click "Clear pull requests" and verify the styled confirmation dialog appears
  • Verify the clear action removes all pinned PRs
  • Check the leaderboard sidebar hover covers full row width on the discoveries page
  • Verify no TypeScript or lint errors
    Fixes Display watchlist Pull Requests tab in a DataTable #666

Before
before

After
after

plind-junior and others added 7 commits April 21, 2026 07:56
@plind-junior
fix: extend leaderboard row hover to full width in sidebar
c4acd7e
@plind-junior
fix: style watchlist clear dialog to match project theme
630a164
@plind-junior
feat: display watchlist Pull Requests tab in a DataTable
5fd5728 
Replace simple WatchedItemRow layout with reusable DataTable component
for the PR tab. Adds columns for PR number, title, repository, author
with avatar, status chip, score, and watchlist star button.
@plind-junior
style: increase PR table row height and remove card padding
a6293b3
@plind-junior
feat: add search, filters, view toggle, rows dropdown, sorting to wat…
58e7bdf 
…chlist PRs tab

- Search box filters by title, repo, author, PR number
- Status filter buttons (All/Open/Merged/Closed) with counts
- List/card view mode toggle matching repositories page style
- Rows per page dropdown (10/25/50) with pagination
- Sortable table columns (PR, title, repo, author, score)
- Card view styled to match MinerCard pattern
@plind-junior
Merge branch 'test' into feat/watchlist-pr-datatable
686562b
@plind-junior
refactor(watchlist): drop redundant JetBrains Mono fontFamily overrides
6971d27 
JetBrains Mono is already the app-wide default via theme.typography.fontFamily,
so per-component declarations have no effect.
@anderdc anderdc added the enhancement New feature or request label Apr 21, 2026
@plind-junior
Copy link
Copy Markdown
Contributor Author

plind-junior commented Apr 23, 2026

@anderdc @e35ventura Ready for review

@e35ventura e35ventura merged commit cd46eba into entrius:test Apr 26, 2026
2 checks passed
@e35ventura e35ventura mentioned this pull request Apr 26, 2026
Closed
12 tasks
jerryzai pushed a commit to jerryzai/gittensor-ui that referenced this pull request Apr 26, 2026
@plind-junior @jerryzai
Feat/watchlist pr datatable (entrius#667)
91c3353
@bitloi bitloi mentioned this pull request Apr 26, 2026
Open
12 tasks
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
26b8329 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs whose hotkey belongs to a watched miner and
PRs whose repository is watched, in addition to explicit stars. Hotkey
is the join key because it is always present on /prs responses while
githubId is only on /miners.

The list/card view-mode toggle, status pills, search, sort, pagination,
and per-row star buttons added in entrius#667 are intentionally untouched.

Closes entrius#784
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
9588dad 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs authored by a watched miner and PRs whose
repository is watched, in addition to explicit stars.

Miner matching uses pr.githubId directly against the watchlist's stored
githubIds. The /prs endpoint always populates githubId, so no /miners
round-trip is needed and matches still resolve when /miners is loading,
stale, or no longer lists a previously-registered miner.

Each row now carries a 'Why' badge (Starred / Miner / Repo) so users
can tell at a glance why a PR is in their watchlist; the badges also
appear on the card-view tile. The DataTable receives isLoading from
useWatchedPRs so the empty-state copy no longer flickers in while
/prs is fetching.

The list/card view-mode toggle, status pills, search, sort,
pagination, and per-row star buttons added in entrius#667 are intentionally
untouched.

Closes entrius#784
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
c2e4057 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs authored by a watched miner and PRs whose
repository is watched, in addition to explicit stars.

Miner matching uses pr.githubId directly against the watchlist's stored
githubIds. The /prs endpoint always populates githubId, so no /miners
round-trip is needed and matches still resolve when /miners is loading,
stale, or no longer lists a previously-registered miner.

Each row now carries a 'Why' badge (Starred / Miner / Repo) so users
can tell at a glance why a PR is in their watchlist; the badges also
appear on the card-view tile. The DataTable receives isLoading from
useWatchedPRs so the empty-state copy no longer flickers in while
/prs is fetching.

The list/card view-mode toggle, status pills, search, sort,
pagination, and per-row star buttons added in entrius#667 are intentionally
untouched.

Closes entrius#784
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
7415ce0 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs authored by a watched miner and PRs whose
repository is watched, in addition to explicit stars.

Miner matching uses pr.githubId directly against the watchlist's stored
githubIds. The /prs endpoint always populates githubId, so no /miners
round-trip is needed and matches still resolve when /miners is loading,
stale, or no longer lists a previously-registered miner.

Each row now carries a 'Why' badge (Starred / Miner / Repo) so users
can tell at a glance why a PR is in their watchlist; the badges also
appear on the card-view tile. The DataTable receives isLoading from
useWatchedPRs so the empty-state copy no longer flickers in while
/prs is fetching.

The list/card view-mode toggle, status pills, search, sort,
pagination, and per-row star buttons added in entrius#667 are intentionally
untouched.

Closes entrius#784
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
fb888d9 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs authored by a watched miner and PRs whose
repository is watched, in addition to explicit stars.

Miner matching uses pr.githubId directly against the watchlist's stored
githubIds. The /prs endpoint always populates githubId, so no /miners
round-trip is needed and matches still resolve when /miners is loading,
stale, or no longer lists a previously-registered miner.

Each row now carries a 'Why' badge (Starred / Miner / Repo) so users
can tell at a glance why a PR is in their watchlist; the badges also
appear on the card-view tile. The DataTable receives isLoading from
useWatchedPRs so the empty-state copy no longer flickers in while
/prs is fetching.

The list/card view-mode toggle, status pills, search, sort,
pagination, and per-row star buttons added in entrius#667 are intentionally
untouched.

Closes entrius#784
bitloi added a commit to bitloi/gittensor-ui that referenced this pull request Apr 27, 2026
@bitloi
feat(watchlist): auto-show PRs from watched miners and repos on PR tab
71d3223 
The Pull Requests tab previously showed only individually starred PRs.
It now also includes PRs authored by a watched miner and PRs whose
repository is watched, in addition to explicit stars.

Miner matching uses pr.githubId directly against the watchlist's stored
githubIds. The /prs endpoint always populates githubId, so no /miners
round-trip is needed and matches still resolve when /miners is loading,
stale, or no longer lists a previously-registered miner.

Each row now carries a 'Why' badge (Starred / Miner / Repo) so users
can tell at a glance why a PR is in their watchlist; the badges also
appear on the card-view tile. The DataTable receives isLoading from
useWatchedPRs so the empty-state copy no longer flickers in while
/prs is fetching.

The list/card view-mode toggle, status pills, search, sort,
pagination, and per-row star buttons added in entrius#667 are intentionally
untouched.

Closes entrius#784
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@e35ventura e35ventura e35ventura approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Display watchlist Pull Requests tab in a DataTable

3 participants

@plind-junior @e35ventura @anderdc