Skip to content

Conversation

conradkleinespel
Copy link
Contributor

Hello !

Thanks for the 2.0 update, it makes a lot of sense, and removes the deprecated Ivory dependency, so good ! 👍

Unfortunately, at the moment, it seems like adding a From address is a hard requirement, even when using templates (which you can set up the From address for in the Sparkpost UI).

This fixes this use case. I didn't dive to deep into the details : maybe there should be some additional checks in case we are not using a template, in which case a From would indeed be needed.

Just wanted to get feedback first.

Best regards,
Conrad @ reassurez-moi.fr

@richleland richleland merged commit 40558e0 into SparkPost:master Jun 29, 2016
@richleland
Copy link
Contributor

Awesome we were just talking about this issue today @conradkleinespel - thanks! I'll make a patch release here in a few minutes.

@conradkleinespel conradkleinespel deleted the from-fix branch June 29, 2016 22:02
@conradkleinespel
Copy link
Contributor Author

@richleland Awesome, thanks for the quick reaction time 👍

@richleland
Copy link
Contributor

np! 2.0.1 is now available.

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.

2 participants