Skip to content

Copy partner ID added#3425

Merged
steven-tey merged 2 commits intomainfrom
copy-partner-id
Feb 6, 2026
Merged

Copy partner ID added#3425
steven-tey merged 2 commits intomainfrom
copy-partner-id

Conversation

@marcusljf
Copy link
Collaborator

@marcusljf marcusljf commented Feb 5, 2026

Adding dropdown functionality to the partner profile to allow for copying partner ID. Split by actions/settings and status changes (destructive).

CleanShot 2026-02-05 at 16 22 03@2x
CleanShot.2026-02-05.at.11.20.09.mp4

Summary by CodeRabbit

  • New Features
    • Copy Partner ID: Users can now quickly copy partner identifiers to clipboard from the partner actions menu for seamless sharing and collaboration
    • Improved Partner Actions Menu: Reorganized with better visual grouping and structure, making it easier to locate and execute specific partner management actions

Adding dropdown functionality to the partner profile to allow for copying partner ID.
@vercel
Copy link
Contributor

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dub Ready Ready Preview Feb 6, 2026 0:30am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

The PR restructures the partner actions popover in the partners dashboard layout, refactoring the action menu from a responsive-width container to a full-width grouped structure with multiple action sections, and introduces new partner management actions (Archive, Reactivate, Deactivate, Ban, Unban) alongside a Copy Partner ID feature.

Changes

Cohort / File(s) Summary
Partner Actions Popover Restructure
apps/web/app/app.dub.co/(dashboard)/[slug]/(ee)/program/partners/[partnerId]/layout.tsx
Refactored popover content structure from responsive grid to full-width container with organized action groups. Added new partner management actions (Archive/Unarchive, Reactivate, Deactivate, Ban/Unban) with conditional rendering based on partner status. Introduced Copy Partner ID action that copies to clipboard and displays toast notification. Added Copy icon import.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • PR #3228 — Overlapping changes to the same partner actions popover structure, invite-delete flow, and partner action grouping.
  • PR #2850 — Modifies the same partners layout file with icon import updates and partner action UI changes.

Suggested reviewers

  • steven-tey

Poem

🐰 Actions grouped with care so bright,
Copy IDs, ban, and archive right!
The popover dances, structured anew—
Partner management flows straight through! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding copy partner ID functionality to the partner action popover.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch copy-partner-id

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Split by partner actions/settings and status changes.
@marcusljf marcusljf marked this pull request as ready for review February 6, 2026 00:23
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In
`@apps/web/app/app.dub.co/`(dashboard)/[slug]/(ee)/program/partners/[partnerId]/layout.tsx:
- Around line 362-371: The click handler for the MenuItem currently calls
navigator.clipboard.writeText(partner.id) without awaiting or handling errors;
update the onClick to be async and wrap the await
navigator.clipboard.writeText(partner.id) in a try-catch, call
toast.success("Partner ID copied!") and setIsOpen(false) only after the awaited
write succeeds, and in the catch call toast.error with a helpful message (e.g.,
"Failed to copy Partner ID") so permission or other errors are surfaced; keep
references to MenuItem, navigator.clipboard.writeText, partner.id, setIsOpen,
and toast.success/toast.error to locate and modify the handler.

@steven-tey steven-tey merged commit 1c55bde into main Feb 6, 2026
10 of 11 checks passed
@steven-tey steven-tey deleted the copy-partner-id branch February 6, 2026 20: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.

2 participants