HTML elements are displayed when editing a comment and adding markdown

[isagoico]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Navigate to a conversation
2. Send a message
3. Edit the message and add a markdown to any of the words

Expected Result:

LHN should show the message preview without any HTML elements

Actual Result:

LHN preview shows HTML elements

Workaround:

No need. Visual issue.

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.0.85-6

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

This only happens when adding markdown when editing messages.

Expensify/Expensify Issue URL:

View all open jobs on GitHub

---

From @puneetlath https://expensify.slack.com/archives/C01GTK53T8Q/p1629137110399300

LHN shows html element if you edit in markdown tags. Doesn’t do it if you put the markdown tag in the original message.

[MelvinBot]

Triggered auto assignment to @cead22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[MelvinBot]

@cead22 Whoops! This issue is 2 days overdue. Let's get this updated quick!

[cead22]

I missed this assignment but I was able to reproduce. Looking to see if this can be sent to external contributors

[parasharrajat]

Proposal

Ok, I have found the issue.
Here this code sets the HTML as the last message text which should be text.

App/src/libs/actions/Report.js

Line 528 in 27b1c52

lastMessageText: message.html,

to as we backend don't send a text.

lastMessageText: Str.stripHTML(htmlForNewComment.replace(/((<br[^>]*>)+)/gi, ' '))

[MelvinBot]

Triggered auto assignment to @NikkiWines (Demolition), see https://stackoverflow.com/c/expensify/questions/8099 for more details.

[cead22]

There's some context here https://expensify.slack.com/archives/C03TQ48KC/p1629763222295200

[NikkiWines]

Don't have a ton of time for this one at the moment with allocations - gonna drop it down to a weekly and hold it for a bit and if I haven't gotten to it by next week I'll ping #eng-chat to see if there's someone who wants to take over.

[NikkiWines]

Gonna work on this today!

[MelvinBot]

This issue has not been updated in over 15 days. @NikkiWines eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!