Skip to content

feat(inbox): Priority sorting, status filter, suggested reviewers#1407

Merged
Twixes merged 1 commit intomainfrom
twixes/inbox-sorting-reviewers
Apr 2, 2026
Merged

feat(inbox): Priority sorting, status filter, suggested reviewers#1407
Twixes merged 1 commit intomainfrom
twixes/inbox-sorting-reviewers

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented Apr 1, 2026

Problem

Inbox sorts by total_weight with no way to filter by status. Cloud sorts by status → suggested reviewer → priority, has a status multi-select, and shows a reviewer indicator on cards. That's the experience Code needs too.

Changes

  • Default ordering: status,-is_suggested_reviewer,priority then toolbar-selected field
  • Add priority as default sort option alongside Strongest signal / Newest / Oldest
  • Replace hardcoded INBOX_PIPELINE_STATUS_FILTER with a persisted status multi-select in the toolbar popover
  • Blue UserIcon badge on report cards when is_suggested_reviewer is true
  • Rename search placeholder to "Search reports..."
  • Add searchDisabledReason prop for disabled state with tooltip

@Twixes Twixes force-pushed the twixes/inbox-signal-cards branch from c4ba030 to 9e66fca Compare April 1, 2026 01:51
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 7279f12 to ea9068f Compare April 1, 2026 01:51
@Twixes Twixes force-pushed the twixes/inbox-signal-cards branch from 9e66fca to 9fd2ea0 Compare April 1, 2026 14:26
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from ea9068f to 5f888c8 Compare April 1, 2026 14:26
@Twixes Twixes changed the title feat(inbox): priority sorting, status filter, suggested reviewers feat(inbox): Priority sorting, status filter, suggested reviewers Apr 1, 2026
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 5f888c8 to 8fbfe0a Compare April 1, 2026 14:55
@Twixes Twixes force-pushed the twixes/inbox-signal-cards branch from 9fd2ea0 to 7b0e7ce Compare April 1, 2026 15:00
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 8fbfe0a to 78671e0 Compare April 1, 2026 15:00
@Twixes Twixes force-pushed the twixes/inbox-signal-cards branch from 7b0e7ce to 37ceb3b Compare April 1, 2026 15:12
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 78671e0 to 24ff688 Compare April 1, 2026 15:12
@Twixes Twixes marked this pull request as ready for review April 1, 2026 15:13
@Twixes Twixes requested review from a team April 1, 2026 15:15
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 24ff688 to d0de55c Compare April 1, 2026 16:52
@Twixes Twixes force-pushed the twixes/inbox-signal-cards branch 2 times, most recently from 10fdf81 to d04184c Compare April 1, 2026 17:08
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from d0de55c to 48cb6bf Compare April 1, 2026 17:08
Copy link
Copy Markdown
Member Author

Twixes commented Apr 2, 2026

Merge activity

  • Apr 2, 1:35 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 2, 1:44 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 2, 1:51 PM UTC: @Twixes merged this pull request with Graphite.

@Twixes Twixes changed the base branch from twixes/inbox-signal-cards to graphite-base/1407 April 2, 2026 13:36
@Twixes Twixes changed the base branch from graphite-base/1407 to main April 2, 2026 13:42
- Default sort: status → suggested reviewer → priority (P0 first)
- Status multi-select filter replacing hardcoded constant
- Priority as default sort option in toolbar
- Blue badge on report cards for suggested reviewer
- Search renamed to "Search reports...", disabled with tooltip when empty
@Twixes Twixes force-pushed the twixes/inbox-sorting-reviewers branch from 48cb6bf to f917d12 Compare April 2, 2026 13:43
@Twixes Twixes merged commit 6ff0105 into main Apr 2, 2026
16 checks passed
@Twixes Twixes deleted the twixes/inbox-sorting-reviewers branch April 2, 2026 13:51
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.

2 participants