I would like to change the way the messages in digest email are stacked together. Right now the messages are divided by "----" string and they don't even start from a new line. So I want to change the way the messages are put together. Can someone point me to the place where I can do that please?
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | 2236179-13.patch | 583 bytes | jhedstrom |
| #8 | message_digest-remove_mail_header-2236179-8.patch | 493 bytes | PawelR |
| #3 | 2236179-bug.jpg | 113.13 KB | dmitrii |
| #1 | message_digest-remove_mail_header-2236179.patch | 534 bytes | chilic |
Comments
Comment #1
chilic commentedit's happens because module set email header = text/html
By default Drupal use email header text/plain.
Comment #2
ohthehugemanatee commentedIf the module wants to support text/html that's fine, but I agree with @Chilic, text/plain should at least be the default. Does this need to be set explicitly by message_digest? Can we expose it as an admin option?
Comment #3
dmitrii commentedSimilar issue with header at Drupal Association site. Maybe related.
Comment #4
BarisW commentedWould it be possible to do it like this?
Comment #5
vali hutchison commentedIf you install the Mime Mail module then it works fine - https://www.drupal.org/project/mimemail (Mime Mail requires Mail System to be installed so you'll need that too - https://www.drupal.org/project/mailsystem)
Install both, enable, and you should get HTML emails sent fine without any further config.
Comment #6
BarisW commentedThanks @ValiH for the tip. Will try!
Comment #7
PawelR commentedI can't see a reason for hard coding 'text/html' in message_digest_mail().
Patch #1 should be committed.
Comment #8
PawelR commentedPatch #1 didn't apply any more.
I've re-rolled it.
Comment #9
chilic commentedThank you @PawelR.
Comment #10
subhojit777RBTC +1
Comment #11
bsarchive commentedI'm using Mime Mail and Message digest isn't picking up my mail.css styling, although message_notify is. ?
Comment #12
tony.herr commentedThe patch in #8 worked for me. Thanks!
Comment #13
jhedstromCorresponding patch for 8.x. I think the 7.x version can also be committed.
Comment #16
jhedstromCommitted to 7 and 8. Thanks!