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

add header support for Email.send #963

Closed
wants to merge 3 commits into from
Closed

Conversation

@spang
Copy link
Contributor

@spang spang commented Apr 18, 2013

I'm gonna want this for an app I'm building soonish.

It's a simple update since mailcomposer has support for custom headers. This branch includes documentation and test updates as well.

spang added 3 commits Apr 17, 2013
Ordering is not preserved since we're using a dictionary, but header
ordering doesn't matter anyway.
I'd like to test that adding multiple headers is working fine too, but
the current tests are already order-dependent. Fixing this properly
would probably involve the tests depending on something like mailparser.
@glasser
Copy link
Member

@glasser glasser commented Apr 26, 2013

Thanks, merged!

@glasser glasser closed this Apr 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants