[openstreetmap/openstreetmap-website] Dressed up notification mails (#1401)

Hervé Saint-Amand notifications at github.com
Sun Feb 5 21:03:01 UTC 2017


Thanks for merging @tomhughes ! There's a couple minor display bugs that have cropped up in Mirosoft mail clients (see https://wiki.openstreetmap.org/wiki/Dressed-up_Notification_Mails) and I'm working on a fix. Should I continue pushing to the same branch, or should I open a new PR for those?

Also I never tested what happens for users who use Gravatar. Best case scenario is the avatar won't display, but worst case is the mailer code will error out and the message won't be sent. I'll check that next.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1401#issuecomment-277549101
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170205/b5d82eb7/attachment.html>


More information about the rails-dev mailing list