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

New User notification emails formatting is broken. #1577

Closed
2 tasks
Benunc opened this issue Mar 14, 2017 · 2 comments
Closed
2 tasks

New User notification emails formatting is broken. #1577

Benunc opened this issue Mar 14, 2017 · 2 comments
Assignees
Milestone

Comments

@Benunc
Copy link
Member

Benunc commented Mar 14, 2017

Issue Overview

If a new user is created in the process of making a donation, the new username/password email is broken formatting.

Expected Behavior

It should send an html email

Current Behavior

screen shot 2017-03-14 at 2 00 09 pm

The html is broken. I think this is related to it sending it in plaintext, but email clients are still trying to parse the html.

Possible Solution

instead of using wp_mail, use the Give_Emails class?

Steps to Reproduce (for bugs)

  1. require login on a form, and submit a donation
  2. check the email sent.
  3. It will be formatted weird

we've had multiple reports of this from users, and I was able to replicate it with mailcatcher on local.

Todos

  • Tests
  • Documentation (developer)
@DevinWalker DevinWalker added this to the 1.8.6 milestone Mar 15, 2017
@Benunc
Copy link
Member Author

Benunc commented Mar 28, 2017

Related: would be nice to send them to a page other than wp-login if possible. Or make that entire message filterable. (user request)

@DevinWalker DevinWalker modified the milestones: 1.8.6, 1.8.7 Mar 31, 2017
@Benunc
Copy link
Member Author

Benunc commented Apr 14, 2017

another request for this today (filterable)

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

3 participants