Skip to content

feat: add Broadcasts.recipients method - #224

Merged
dielduarte merged 5 commits into
mainfrom
feat/broadcasts-recipients
Aug 24, 2026
Merged

feat: add Broadcasts.recipients method#224
dielduarte merged 5 commits into
mainfrom
feat/broadcasts-recipients

Conversation

@dielduarte

@dielduarte dielduarte commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Adds Resend::Broadcasts.recipients(id, params) for GET /broadcasts/{id}/recipients.

Spec: resend/resend-openapi#97
Node reference: resend/resend-node#1078

Tests: 357 passed. rubocop clean.


Summary by cubic

Adds Resend::Broadcasts.recipients(id, params) to list broadcast recipients via GET /broadcasts/{id}/recipients. Previously the SDK could not fetch recipients; now clients can filter by event type and paginate, and 404s raise Resend::Error::NotFoundError. Removes the recipients example that required an arbitrary sleep.

  • Require params[:type] (sent, delivered, opened, clicked, bounced, complained, unsubscribed, suppressed); raise ArgumentError if missing.
  • Support filters: email; bounce_type when type is 'bounced' (permanent, transient, undetermined); limit, after, before via Resend::PaginationHelper.
  • Return extra fields by type: opened includes count; clicked includes count and clicked_links; bounced includes bounce_type.
  • Tests verify request paths, per-type fields, and that 404 raises NotFoundError.

Written for commit b6c8b67. Summary will update on new commits.

Review in cubic

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

Copy link
Copy Markdown
Contributor

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 3 files

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

Re-trigger cubic

Comment thread spec/broadcasts_spec.rb
Comment thread examples/broadcasts.rb Outdated
Comment thread spec/broadcasts_spec.rb

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

0 issues found across 2 files (changes from recent commits).

Confidence score: 5/5

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

Requires human review: Auto-approval blocked by 2 unresolved issues from previous reviews.

Re-trigger cubic

@dielduarte
dielduarte marked this pull request as ready for review August 22, 2026 02:02

@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 the recipients method to the Broadcasts module, allowing users to fetch recipients for a specific broadcast with support for event type filtering and pagination. The implementation follows standard SDK patterns.

Re-trigger cubic

@github-actions github-actions Bot added the linear-synced PR has been synced to Linear label Aug 22, 2026
@dielduarte
dielduarte merged commit bc3a075 into main Aug 24, 2026
8 checks passed
@dielduarte
dielduarte deleted the feat/broadcasts-recipients branch August 24, 2026 15:45
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