[$125] Chat - With quote markdown, url text on tapping enter key not moved to next line.

[IuliiaHerets]

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

---

Version Number: 9.0.78-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
Issue reported by: Applause Internal Team
Device used: Redmi note 10s Android 13
App Component: Chat Report View

Action Performed:

1. Go to https://staging.new.expensify.com/home
2. Open chat
3. Enter > jsjjd.com
4. Send the message
5. Edit the message
6. Tap enter key and try to make url text into two lines

Expected Result:

With quote markdown, url text on tapping enter key must be moved to next line.

Actual Result:

With quote markdown, url text on tapping enter key not moved to next line.

Workaround:

Unknown

Platforms:

• Android: Standalone
• Android: HybridApp
• Android: mWeb Chrome
• iOS: Standalone
• iOS: HybridApp
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Bug6701644_1735017397193.Screenrecorder-2024-12-24-10-40-53-895_compress_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021871675231934377672
• Upwork Job ID: 1871675231934377672
• Last Price Increase: 2024-12-31
• Automatic offers:
• daledah | Contributor | 105551577

Issue Owner

Current Issue Owner: @sakluger

[melvin-bot]

Triggered auto assignment to @sakluger (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[sakluger]

I tested myself in the iOS app and I saw different behavior from the video above. I tested with hshs.com and google.com.

With hshs.com, it kept everything on the same line:

With google.com, which was too big to have on a single line, it kept the https:// on the first line and moved the remainder to the second line:

I also tested on MacOS chrome and could not reproduce.

[sakluger]

I'm going to set the bounty to $125 to start, hopefully this is a simple fix.

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Upwork job price has been updated to $125

[daledah]

Edited by proposal-police: This proposal was edited at 2024-12-26 01:40:39 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

With quote markdown, url text on tapping enter key not moved to next line.

What is the root cause of that problem?

We have the logic to throw the error if text !== markdown

https://github.com/Expensify/react-native-live-markdown/blob/6447ea9ec54830f1d55c1eac0e9274ce93a4c769/src/parseExpensiMark.ts#L283-L289

What changes do you think we should make in order to solve the problem?

This PR removes try/catch so when there's parse error, it cause the issue above. I think we should bring try/catch back.

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

We should create the test for parseExpensiMark to cover the case when there's an parse error

What alternative solutions did you explore? (Optional)

We can instead show the warning and return empty array

if (text !== markdown) {
  console.warn(
    `[react-native-live-markdown] Parsing error: the processed text does not match the original Markdown input. This may be caused by incorrect parsing functions or invalid input Markdown.\nProcessed input: '${JSON.stringify(
      text,
    )}'\nOriginal input: '${JSON.stringify(markdown)}'`,
  );
  return [];
}

Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.