Skip to content

Add line breaks to outgoing mails

Dominik Hayon requested to merge pfaufisch-display-line-breaks-in-mails into master

What does this MR do?

As mentioned here, outgoing mails are not displaying line breaks. This MR adds a nl2br fo fix this.

How confident are you it won't break things if deployed?

Very

Links to related issues

Any relevant links (issues, documentation, slack discussions).

How to test

Steps a reviewer can take to verify that this MR does what it says it does e.g.

  1. Checkout branch locally
  2. Login as bot
  3. Write mail containing line breaks
  4. Check if mail is correctly displayed in outgoing mails

Screenshots (if applicable)

Before:

before

After:

after

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Chris Oelmueller

Merge request reports