Show extended information when a new comment is in a digest email#12563
Show extended information when a new comment is in a digest email#12563
Conversation
|
Failing spec is fixed with #12566 |
alecslupu
left a comment
There was a problem hiding this comment.
@andreslucena , i have tried to check this PR, but i think you forgot to attach the preview mailer class.
When trying to verify the reproduction steps, i could not access the preview class due to error.
Oopsie, I've added it in 8eed44d |
alecslupu
left a comment
There was a problem hiding this comment.
The HTML version is displaying correctly. On the text version in multiple mailers we have an arrow --> that seems that an html comment may have been wrongly truncated. ( not in scope of this PR )
|
@andreslucena , can you merge with the latest develop so that we validate there is no issue with the pipeline ? |
Done in 60932cc, everything is green 💚 |
…2563) * Show extended information when a new comment is in a digest email * Add newsletter preview mailer class
🎩 What? Why?
When having the digest email for notifications and there's activity with comments, you get always the same notification. I saw this the past week with some activity in Metadecidim:
This PR fixes that by adding the comment body to these emails.
📌 Related Issues
Testing
📷 Screenshots