Expensify.Cash - Markdown formatting removed after editing a message

[sakluger]

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

---

Expected Result:

Any formatting that was added to a message via markdown should remain after the message is edited. To do this, we should show the original markdown in the "Edit message" text field and apply that markdown after saving the changes.

Actual Result:

After editing a message, all markdown formatting (such as bold, code blocks, etc) is removed. If I try to add it back, the markdown just shows as text.

Action Performed:

1. Log into e.cash on web or on desktop
2. Send a message to anyone with any markdown (* on each side of the text will make the text bold)
3. Confirm that the message formatting has been applied correctly.
   
4. Right-click on the message you just sent and select Edit Message
   
5. Click Save Changes
   
6. Confirm that formatting has been removed
   
7. Edit the same message again and try adding back the original markdown
8. Confirm that the markdown characters save as text
   

Workaround:

The only workaround is to send a new message with the intended markdown.

Platform:

Where is this issue occurring?

Web ✔️
iOS
Android
Desktop App ✔️
Mobile Web

Version Number: Version 1.0.51-0
Logs: No logs
Notes/Photos/Videos: Screenshots are above in the "Action Performed" section
Expensify/Expensify Issue URL: None

View all open jobs on Upwork

[MelvinBot]

Triggered auto assignment to @jboniface (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[kakajann]

I saw this bug before and I was going to create an issue.

Here's why this happening:

When you send message addAction function will be called and it parses the message like this

But when you edit a message editReportComment function will be called and it doesn't parse the message.

If you export this issue to upwork, I can work on this.

[MelvinBot]

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

[MelvinBot]

Triggered auto assignment to @SofiedeVreese (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[MelvinBot]

Triggered auto assignment to @MitchExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[jasperhuangg]

Yo @MitchExpensify! @kakajann expressed interest on working on this and seems to have a solution in mind. Let me know if I need to review any proposals!

[MitchExpensify]

Cool, @jasperhuangg! Creating UpWork job now

[MitchExpensify]

Here is the Job posting if you'd like to apply for it @kakajann! https://www.upwork.com/jobs/~01a7eb95dc405570e3

[MelvinBot]

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[roryabraham]

This issue is a duplicate of #2847, which is already being worked on. Closing this out! @MitchExpensify please don't hire anyone on Upwork for this.

[roryabraham]

Also, just FYI, there is a tracking issue that this will eventually fall under here.