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

Allow passing of a headers object to be included in Email.send #2854

Closed
wants to merge 2 commits into from

Conversation

@stefankutko
Copy link

@stefankutko stefankutko commented Oct 18, 2014

This is particularly useful when sending via a third-party email provider (such as SendGrid or Mandrill) and you want to specify a template or other options.

@apollo-cla
Copy link

@apollo-cla apollo-cla commented Oct 18, 2014

@stefankutko: Before we can merge your pull request, you'll need to sign the Meteor Contributor Agreement: https://contribute.meteor.com/

@glasser
Copy link
Member

@glasser glasser commented Dec 3, 2014

This doesn't seem like the best API. It seems like this should be on the emailTemplates object along with the other information about the email. That way it will work with accounts-ui too. How about if the emailTemplates sub-objects get a function headers alongside subject/text/html?

Please also provide tests (see accounts-password/email_tests.js).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.