Skip to content

feat: add broadcasts->recipients() method - #141

Merged
jayanratna merged 2 commits into
mainfrom
feat/broadcasts-recipients
Aug 26, 2026
Merged

feat: add broadcasts->recipients() method#141
jayanratna merged 2 commits into
mainfrom
feat/broadcasts-recipients

Conversation

@dielduarte

@dielduarte dielduarte commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Adds Broadcast::recipients() for GET /broadcasts/{id}/recipients.

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

Tests: 193 passed.

Note: extends the shared query-param allowlist in Payload::list() to permit type/email/bounce_type — used by every list endpoint in this SDK, worth a look.


Summary by cubic

Adds Broadcast::recipients() to list recipients for GET /broadcasts/{id}/recipients, enabling filtered, paginated analytics by event type. Previously the SDK had no helper and dropped type, email, and bounce_type; now these filters work and are scoped to this endpoint only.

  • Method: recipients(string $id, array $options) requires type; supports email, bounce_type, limit, before, after; returns Collection<Broadcasts\Recipient>.
  • Adds Broadcasts\Recipient resource and maps 'broadcast-recipients' in Service.
  • Updates Payload::list() to accept endpoint-specific allowlists; only Broadcast::recipients() permits type, email, bounce_type. Other list endpoints are unchanged.

Written for commit 0ef99cf. 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 6 files

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

Re-trigger cubic

Comment thread src/Service/Broadcast.php
Comment thread src/ValueObjects/Transporter/Payload.php Outdated

@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 2 files (changes from recent commits).

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:05
@github-actions github-actions Bot added the linear-synced PR has been synced to Linear label Aug 22, 2026
@jayanratna

Copy link
Copy Markdown
Collaborator

Hey @dielduarte, thank you for creating this. Could you please resolve the merge conflicts?

@dielduarte
dielduarte requested a review from jayanratna August 25, 2026 17:37
Retrieves the recipients of a broadcast for a given event type
(sent, delivered, opened, clicked, bounced, complained, unsubscribed,
suppressed), with optional email and bounce_type filters and cursor
pagination.
@dielduarte
dielduarte force-pushed the feat/broadcasts-recipients branch from 0ef99cf to 2b1133e Compare August 26, 2026 01:57
@jayanratna
jayanratna merged commit 8a0912b into main Aug 26, 2026
8 checks passed
@jayanratna
jayanratna deleted the feat/broadcasts-recipients branch August 26, 2026 03:15
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.

2 participants