[HOLD #140] [$500] Chat - Altering 1st code block removes markdown applied for quote & 2nd code block

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: 1.4.42
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4319035
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1. Launch app
2. Tap on a report
3. Type # 567
4. In next line, type ||| |||
5. Now from 1st pair- remove 5th block tick
6. Note, removing 5th block tick in 1st pair, markdown applied for 2nd pair & for quote message also removed

Expected Result:

Removing 5th block tick in 1st pair, markdown applied for 2nd pair & for quote message must not be removed

Actual Result:

A quote message paired with 2 code block message is send. Altering 1st code block changes
quote and 2nd code block message

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6381173_1708027679048.cod.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0194d1ba71ece129a2
• Upwork Job ID: 1758228337378869248
• Last Price Increase: 2024-02-15

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~0194d1ba71ece129a2

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @hoangzinh (External)

[melvin-bot]

Triggered auto assignment to @Christinadobrzyn (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[lanitochka17]

We think that this bug might be related to #vip-vsp
CC @quinthar

[Christinadobrzyn]

I can reproduce this but I don't think this is a priority unless there is some markdown reason for us to work on this now.

@hoangzinh what do you think? Should this be prioritised at this time?

[hoangzinh]

@Christinadobrzyn I think we can track this markdown preview issue here #36071 and put it On Hold like this issue #36269. What do you think?

[tomekzaw]

Duplicate of Expensify/react-native-live-markdown#179, fixed in Expensify/react-native-live-markdown#140, will be ported to Expensify/App in a separate PR.

[Christinadobrzyn]

Okay it sounds like we should hold for Expensify/react-native-live-markdown#140 and if that doesn't resolve it we'll add to #36071 and add another hold.

[Christinadobrzyn]

Hold for Expensify/react-native-live-markdown#140

[Christinadobrzyn]

This appears to be fixed with Expensify/react-native-live-markdown#140

2024-03-07_14-01-26.mp4

Closing!