Skip to content

fix: correct email batch api call with reply_to#460

Merged
joaopcm merged 5 commits intocanaryfrom
fix/emails-batch-reply-to-format
Jan 23, 2025
Merged

fix: correct email batch api call with reply_to#460
joaopcm merged 5 commits intocanaryfrom
fix/emails-batch-reply-to-format

Conversation

@joaopcm
Copy link
Member

@joaopcm joaopcm commented Jan 23, 2025

Closes #421

Using this implementation ↓

74614

We were getting this behavior when replying an email ↓

CleanShot 2025-01-22 at 22 00 10@2x

But it should reply to delivered@resend.dev instead.

After fixing it, this is what we get ↓

CleanShot 2025-01-22 at 22 01 14@2x

Correctly mapping the Node.js SDK's fields to Resend's API so we do not forget any option in the payload.

-  update version from 4.1.1 to 4.2.0
-  fix /emails/batch api call to use reply_to instead of replyTo
-  refactor email transformation using parseEmailToAPIOptions in batch and emails
-  export Attachment interface in create-email-options.interface.ts
@joaopcm joaopcm requested a review from a team as a code owner January 23, 2025 00:58
@joaopcm joaopcm self-assigned this Jan 23, 2025
-  changed version from 4.2.0 to 4.1.1 in package.json
-  updated type import in batch.ts
-  modified interface name in email-api-options.interface.ts
-  adjusted function return type in parse-email-to-api-options.ts
…eEmailToApiOptions

-  updated function name in imports across multiple files
-  modified function name in test cases
-  ensured consistent naming convention throughout the codebase
-  change describe block to use correct camelCase for function name
@joaopcm joaopcm requested a review from bukinoshita January 23, 2025 01:16
@joaopcm joaopcm merged commit 594de69 into canary Jan 23, 2025
8 checks passed
@joaopcm joaopcm deleted the fix/emails-batch-reply-to-format branch January 23, 2025 19:35
@JamesSingleton
Copy link

It would be nice to update the docs, everything still references reply_to and even the AI says to use reply_to.
Screenshot 2025-02-13 at 20 42 31

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.

[v4] replyTo is not functioning when using batch

4 participants