New line / Markdown not maintained in incidents and maintenance

[labolanimichele]

Hello,

sorry to reopen this one, but on the latest pull of 2.4 it still does not work for me (pulled today).

Please see attached the template, what is shown in the status page (which is perfect) and what I receive by email (which is not working for me).

Any hint? It may be something trivial I cannot see...
This is currently the only thing which is preventing us from using Cachet for our customer notifications., which is a pity!

Thank you,
Michele

[labolanimichele]

(I had already opened it in #2712, but I still see the same problem)

[jbrooksuk]

Looks like the emails don't support Markdown.

[nalysius]

I read the Laravel documentation and I saw that the message had a markdown method. Perhaps we could use it in app/Notifications/**/*Notification.php instead of line?

[jbrooksuk]

This should be fixed now.

[uxen-ab]

@jbrooksuk I think this issue can be reopened, d63b9b2 seems don't fix it.
This issue was fixed, but by removing the line markdown in 0c96e67 the markdown is no longer interpreted and all the text is on one line, as pointed on Slack by sontt.

[jbrooksuk]

@jbrooksuk I think this issue can be reopened, d63b9b2 seems don't fix it.

@uxen-ab what do you mean it's not fixed, sorry? We should be stripping out all HTML/Markdown.

[uxen-ab]

@jbrooksuk I have not experienced this issue, it's sontt, on Slack, that reported it to me, with screenshots. During the week on Slack I'll ask him to explain more here.
He sent me a screenshot where its email was in this issue, markdown not interpreted and line breaks either.

[sontt710]

Hi,

I created an incident to notification via email, but in an email no new line break, but dashboard work fine.

[jbrooksuk]

This will be fixed in #3100