Skip to content

feat: show PR status icons on workspace cards in sidebar (Refs: beans-f05i)#145

Merged
hmans merged 1 commit into
mainfrom
beans/limited-capybara-tzl5
Mar 17, 2026
Merged

feat: show PR status icons on workspace cards in sidebar (Refs: beans-f05i)#145
hmans merged 1 commit into
mainfrom
beans/limited-capybara-tzl5

Conversation

@hmans
Copy link
Copy Markdown
Owner

@hmans hmans commented Mar 17, 2026

Summary

  • In integrate: pr mode, workspace cards in the sidebar now display PR status icons instead of the generic green checkmark:
    • No PR: no icon shown
    • Checks pending: orange clock icon
    • Checks failed: red X circle icon
    • Checks passed: green check circle icon
    • PR merged: purple rotated branch icon (matching the existing PR button style)
  • Icons are clickable links to the PR URL
  • On hover, the destroy (archive) button still appears
  • In integrate: local mode, behavior is completely unchanged

Test plan

  • Verify icons appear correctly for workspaces with PRs in integrate: pr mode
  • Verify each status (pending, fail, pass, merged) shows the correct icon and color
  • Verify clicking the icon opens the PR URL
  • Verify hover shows the destroy button
  • Verify integrate: local mode still shows the green checkmark as before
  • Verify workspaces without a PR show no icon (just the hover destroy button)

- In `integrate: pr` mode, workspace cards now show PR status icons:
  - Merged: purple rotated branch icon
  - Checks failed: red X circle
  - Checks pending: orange clock
  - Checks passed: green check circle
  - No PR: no icon (falls through to existing logic)
- Icons link to the PR URL and show destroy button on hover
- `integrate: local` mode behavior unchanged (green checkmark)

Refs: beans-f05i
@hmans hmans merged commit 482fa78 into main Mar 17, 2026
1 check passed
@hmans hmans deleted the beans/limited-capybara-tzl5 branch March 17, 2026 19:16
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