Skip to content

feat(web): copy PR links from multi-selected sidebar threads - #71

Draft
wSedlacek wants to merge 1 commit into
mainfrom
t3/9660d431
Draft

wSedlacek wants to merge 1 commit into
mainfrom
t3/9660d431

Conversation

@wSedlacek

Copy link
Copy Markdown

Reopened from closed upstream PR pingdotgg#30713 as a draft for t3.1code.
Idea rank: 70 of 108. Branch is as it was upstream; it likely needs a rebase (modernize) before review.

Summary

  • Multi-select in the sidebar could settle/snooze threads, but there was no way to copy every linked PR URL at once.
  • Adds Copy PR links (N) to the multi-select context menu; unique visible PR URLs go to the clipboard as newline-separated text.
  • Hides the action when none of the selected threads have linked PRs; selection stays so you can settle afterward.

Test plan

  • vp test run apps/web/src/components/Sidebar.logic.test.ts -t "collectVisiblePullRequestUrls|buildBulkCopyPullRequestLinks"
  • vp run check:changed
  • Multi-select threads with linked PRs → right-click → Copy PR links → paste shows unique URLs
  • Multi-select threads with no PRs → menu has no Copy PR links item
  • Stack-dismissed PRs are not included

Made with Cursor

🤖 Generated with Claude Code

Reopened from pingdotgg#30713 and squashed onto current main.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted Author is trusted by repo permissions or the VOUCHED list. size:L labels Nov 14, 2026

This branch has not been deployed

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

Labels

size:L vouch:trusted Author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant