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

[5.2] Added reply_to field for SparkPost API request within SparkpostTransport #13410

Merged

Conversation

byronsmith
Copy link
Contributor

Added reply_to field for SparkPost API request within SparkpostTransport

Signed-off-by: Byron Smith byronsmith/framework@30db80bfe262

Signed-off-by: Byron Smith <byronsmith@30db80bfe262>
@byronsmith byronsmith force-pushed the feature/sparkpost-transport-replyto branch from 8a4d104 to 62d4fc7 Compare May 3, 2016 04:18
@GrahamCampbell GrahamCampbell changed the title Added reply_to field for SparkPost API request within SparkpostTransport [5.2] Added reply_to field for SparkPost API request within SparkpostTransport May 3, 2016
@GrahamCampbell
Copy link
Member

What about the other drivers please?

@byronsmith
Copy link
Contributor Author

Happy to contribute to the Mailgun transport, but mandrill is no longer a free service, and I don't have a EC2 instance for SES. I therefore wouldn't be comfortable implementing code I cannot test.

With that in mind, where does that leave me in terms of having this SparkPost improvement merged?

@taylorotwell taylorotwell merged commit 00ad6ed into laravel:5.2 May 4, 2016
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.

3 participants