[$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 [email protected] 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.

[daledah]

I'm asking for the author the reason why he removed try/catch block: https://github.com/Expensify/react-native-live-markdown/pull/439/files#r1897091802

[tomekzaw]

Responded here: Expensify/react-native-live-markdown#439 (comment)

[daledah]

Thanks @tomekzaw. @getusha What do you think?

[melvin-bot]

@sakluger, @getusha Huh... This is 4 days overdue. Who can take care of this?

[sakluger]

I think @getusha is out on vacation. Let's hold until the new year.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[melvin-bot]

@sakluger, @getusha Still overdue 6 days?! Let's take care of this!

[sakluger]

@getusha welcome back! Let me know if you need to have some of your assigned issues redistributed, I'm happy to assign this to someone else. Otherwise, I'll wait for your review.

[getusha]

@sakluger will review in short. happy new year!

[getusha]

@daledah's proposal looks good to me
🎀 👀 🎀 C+ Reviewed!

[melvin-bot]

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

[melvin-bot]

📣 @daledah 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[tomekzaw]

PR Expensify/react-native-live-markdown#593 has been merged into react-native-live-markdown.

[melvin-bot]

@sakluger @thienlnam @getusha @daledah this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[melvin-bot]

@sakluger, @thienlnam, @getusha, @daledah Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[melvin-bot]

This issue has not been updated in over 15 days. @sakluger, @thienlnam, @getusha, @daledah 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!

[daledah]

@thienlnam FYI Newer version of react-native-live-markdown is merged in this PR so I closed the current PR.

[daledah]

@thienlnam Are there any problems left before we can process payment here?

[thienlnam]

Nope, looks like this has passed the 7 day prod regression period.

cc @sakluger Could you handle payment for this issue? Looks like the automation failed here

[daledah]

@sakluger Friendly bump in case you missed this issue.

[sakluger]

Oops, sorry - I missed this because it was Monthly. Handling now.

[sakluger]

@daledah the old Upwork offer expired, here's the new one: https://www.upwork.com/nx/wm/offer/106378988

@getusha was there a PR that you reviewed for this one?