Skip to content

feat: add broadcasts.ClickedLinks() method - #153

Merged
dielduarte merged 1 commit into
mainfrom
feat/broadcasts-clicked-links-method
Aug 24, 2026
Merged

dielduarte merged 1 commit into
mainfrom
feat/broadcasts-clicked-links-method

Conversation

@dielduarte

@dielduarte dielduarte commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Adds GET /broadcasts/{id}/clicked-links — paginated list of a broadcast's clicked links (url, clicks, unique_clicks).

Spec: resend/resend-openapi#95
Reference implementation: resend/resend-node#1077


Summary by cubic

Adds GET /broadcasts/{id}/clicked-links to fetch a broadcast’s clicked links with pagination. Previously the client could not fetch clicked links; now it supports per-link analytics and validates empty broadcastId.

  • Defines ListBroadcastClickedLinksResponse and BroadcastClickedLink with id (opaque pagination cursor), url, clicks, and unique_clicks.
  • Adds BroadcastsSvc methods: ClickedLinks, ClickedLinksWithContext, and ClickedLinksWithOptions; pagination via ListOptions (limit, after); returns "[ERROR]: broadcastId cannot be empty" if missing.
  • Tests cover default retrieval, pagination query params, and validation.
  • Migration: if you implement BroadcastsSvc, add the three new methods.

Written for commit 98756e2. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread broadcasts_test.go

@klotty klotty left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

0 issues found across 1 file (changes from recent commits).

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Auto-approved: Adds support for retrieving broadcast clicked links via the new ClickedLinks method, including pagination and unit tests.

Re-trigger cubic

@dielduarte
dielduarte marked this pull request as ready for review August 22, 2026 01:45
@github-actions github-actions Bot added the linear-synced PR has been synced to Linear label Aug 22, 2026
@dielduarte
dielduarte force-pushed the feat/broadcasts-clicked-links-method branch from 665f7bd to 98756e2 Compare August 24, 2026 13:50
@dielduarte
dielduarte merged commit 95905fe into main Aug 24, 2026
9 checks passed
@dielduarte
dielduarte deleted the feat/broadcasts-clicked-links-method branch August 24, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

linear-synced PR has been synced to Linear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants