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

Line breaks aren't coming through #8

Open
Japh opened this issue Jan 28, 2015 · 7 comments
Open

Line breaks aren't coming through #8

Japh opened this issue Jan 28, 2015 · 7 comments

Comments

@Japh
Copy link

Japh commented Jan 28, 2015

Looking at an email from before I installed the plugin, and one after, the line breaks don't seem to be coming through.

Screenshot of Issue in GMail

@Japh
Copy link
Author

Japh commented Jan 28, 2015

For reference, these emails were generated by the "Contact Form" component in Jetpack.

@danielbachhuber
Copy link
Owner

It's probably because the emails are being sent as HTML through Mandrill, and don't have an explicit content type header.

Can you do a bit of debugging?

@espellcaste
Copy link

Anyone can confirm if the problem still persists as the proper change was added?

@jmslbam
Copy link

jmslbam commented Jun 1, 2015

Hi @espellcaste,

I just downloaded the latest version and stumbled on I think the same issue. Because this PR got overwritten by f3de79d#diff-844352cc65e871bf277f934b5ea2ba7bL108. this fix became obsolete.
To send your e-mails as html add this filter to set the content_type to text/html : https://codex.wordpress.org/Plugin_API/Filter_Reference/wp_mail_content_type

My Gravity Forms first came in as text, now with this filter active, they a displayed properly.

@espellcaste
Copy link

Thanks for the response @jmslbam, but that doesn't mean all emails are going to be sent as html, when the fix your mention recommends not to?

@espellcaste
Copy link

I mean, I see the wp_mail_content_type() filter as a "global" change and not as a local change inside a custom function sending a particular email.

I've been setting the header as html for now inside my custom functions (sending a particular customized email) but haven't tested yet your suggestion.

@owaincuvelier
Copy link

I ran into this today with the password reset link was being displayed as
/domain.dev/wordpress/wp-login.php?action=rp&key=SgzKAGAQDREng4ihnzQv&login=admin>

It worked previously and git bisected to this commit. Hope this helps (if related)

ec8bfa8

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

No branches or pull requests

5 participants