Trim newlines from message previews.

[pixlwave]

• Fixes a bug with replies where the message shows on the line after the sender's name.

This is the simple fix although I we should discuss whether it would make more sense to strip markdown from the preview by building the attributed string from the formatted body to get a clean string instead.

Before	After

[github-actions]

	Warnings
⚠️	Please add a changelog.
⚠️	Some of the commits are missing ticket numbers. Please consider squashing all commits that don't have a tracking number.

Generated by 🚫 Danger Swift against f3ecf63

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +0.26 🎉

Comparison is base (98cec56) 41.11% compared to head (ad4c5ea) 41.37%.

❗ Current head ad4c5ea differs from pull request most recent head f3ecf63. Consider uploading reports for the commit f3ecf63 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1208      +/-   ##
===========================================
+ Coverage    41.11%   41.37%   +0.26%     
===========================================
  Files          413      410       -3     
  Lines        26459    26305     -154     
  Branches     13683    13621      -62     
===========================================
+ Hits         10879    10885       +6     
+ Misses       15286    15126     -160     
  Partials       294      294              

Flag	Coverage Δ
unittests	23.39% <0.00%> (+0.14%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...ices/Room/RoomSummary/RoomEventStringBuilder.swift	4.61% <0.00%> (-0.08%)	⬇️

... and 12 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[stefanceriu]

lgtm 👍

we should discuss whether it would make more sense to strip markdown from the preview by building the attributed string from the formatted body to get a clean string instead

It makes sense to me but it's a product decision in the end

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication