Skip to content

feat(repo): enhance PR discovery with persistent shareable search filters#1033

Open
carlh7777 wants to merge 1 commit into
entrius:testfrom
carlh7777:feat/repo-pr-table-search
Open

feat(repo): enhance PR discovery with persistent shareable search filters#1033
carlh7777 wants to merge 1 commit into
entrius:testfrom
carlh7777:feat/repo-pr-table-search

Conversation

@carlh7777
Copy link
Copy Markdown
Contributor

Summary

Adds URL-backed search to the repository PR table with multi-term AND filtering and persistent, shareable query state.

Users

  • Search by PR title, PR number, merged date, and optional score-style clauses
  • Press Enter to pin multiple terms with AND matching
  • Share or reopen the URL to restore the exact filtered view

Product

  • Search state persists across refreshes and direct links
  • Improves reproducibility and collaboration with shareable filtered views

Engineering

  • Centralized filtering/search helpers in src/utils/prTable.ts
  • Integrated searchable table UI in RepositoryPRsTable.tsx

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Screenshots

Before

image

After

gorec-2026-05-11-05-51-53.webm

Checklist

  • New components are modularized/separated where sensible
  • Uses predefined theme (e.g. no hardcoded colors)
  • Responsive/mobile checked
  • Tested against the test API
  • npm run format and npm run lint:fix have been run
  • npm run build passes
  • Screenshots included for any UI/visual changes

@xiao-xiao-mao xiao-xiao-mao Bot added the enhancement New feature or request label May 11, 2026
@ventura-oss
Copy link
Copy Markdown
Contributor

Skipping in this review pass — the branch is 13 commits behind test and the diff against current test is far larger than the stated change (28 files, +1406/-1594, vs. the described 2-file search-filter addition). Merging in the current state would silently revert recent fixes including:

Please rebase onto current origin/test so the diff reflects only the PR-table search/filter feature, and we'll re-review.

@carlh7777 carlh7777 force-pushed the feat/repo-pr-table-search branch from b6fafce to 110eddf Compare May 13, 2026 11:36
@carlh7777
Copy link
Copy Markdown
Contributor Author

Hi @ventura-oss

Rebased onto the latest origin/test and force-pushed the branch. The PR diff should now be limited to the PR-table search/filter changes only.

@carlh7777 carlh7777 force-pushed the feat/repo-pr-table-search branch 2 times, most recently from 4c1ad51 to 141a9e1 Compare May 14, 2026 08:18
@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 15, 2026

fix conflicts

@carlh7777 carlh7777 force-pushed the feat/repo-pr-table-search branch from 141a9e1 to ed4c852 Compare May 15, 2026 21:57
@carlh7777
Copy link
Copy Markdown
Contributor Author

Hi @anderdc fixed conflicts

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants