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

Send emails as plaintext #273

Closed
The-Compiler opened this issue May 21, 2014 · 3 comments · Fixed by #297
Closed

Send emails as plaintext #273

The-Compiler opened this issue May 21, 2014 · 3 comments · Fixed by #297

Comments

@The-Compiler
Copy link

When receiving a mail from freedomsponsors, I get a multipart/alternative mail, with a text/plain and a text/html part. However, the plain text part is not plain at all:

<!DOCTYPE html> 
<html lang="en"><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
 </head>
  <body style="color: #333;font-size:13px;font-family: 'Helvetica Neue', Helvetica, Arial, sans-serif;margin-bottom: 0px;margin-left: 20px;margin-right: 0px;margin-top: 0px;padding-bottom: 40px;padding-top: 60px;">
    <div style="padding-left: 120px;padding-right: 20px;">

Hello The Compiler<br>
...

I'd suggest to either send actual plaintext in the plaintext part, or send no multipart at all, and just text/html mails instead (then in my case it'd be converted into text by a text browser [elinks] for me to view in my commandline mail client [mutt]).

@tonylampada
Copy link
Member

or send no multipart at all, and just text/html mails instead

I like that. I don't think html emails are necessary indeed.

@tonylampada tonylampada changed the title Plaintext mails are sent as HTML Send emails as plaintext Jul 9, 2014
@sim6
Copy link
Contributor

sim6 commented Jul 17, 2014

I also prefer only plain text emails. But, anyway, please, could you take a look at #297 ?

@tonylampada
Copy link
Member

Merged and deployed. Thanks @sim6!
@The-Compiler, please let me know if this doesn't fix it for you!

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 a pull request may close this issue.

3 participants