Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

F/reply to multiple #1302

Closed
wants to merge 8 commits into from
Closed

Conversation

subinoy7
Copy link
Contributor

Implementation of #339 & #1301
The replyToList support was added in v3 API but it wasn't supported in the library. With this changes, we can:

  • Use the replyToList header to set multiple emails in the replyTo
  • Error handling are placed if the format is not in acceptable
  • Fixed old test cases that were failing as SendGrid changed the acceptable response code to 202

Acceptable Format:
[ { "name": "Test Team", "email": "[email protected]" }, { "name": "Support Test Team", "email": "[email protected]" } ]

If any one of the records doesn't have email filed, then it will return an error. Also the value of replyToList should be an array.

@subinoy7 subinoy7 closed this Sep 23, 2021
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.

1 participant