App: [HOLD for payment 2023-02-20] [$1000] Link syntax doesn't work on link ended with question mark

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. Open staging.new.expensify.com
  2. Open a chat and send this : [link example]([example.com](http://example.com/)?)

Expected Result:

Shown link should be link example with link directed to example.com?

Actual Result:

Link syntax doesn’t work and the shown text is link example

Workaround:

unknown

Platforms:

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

  • MacOS / Chrome / Safari

Version Number: v1.2.62-1 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/215828089-6263aca8-7baa-4899-b64b-1ef0a90fc196.mp4

https://user-images.githubusercontent.com/43996225/215828121-5e2d5f15-ebf3-44c1-9a05-61303913a474.mp4

Expensify/Expensify Issue URL: Issue reported by: @kerupuksambel Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1675115794046579

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01aaa03035e42d0136
  • Upwork Job ID: 1622406207408267264
  • Last Price Increase: 2023-02-06

About this issue

  • Original URL
  • State: closed
  • Created a year ago
  • Comments: 42 (26 by maintainers)

Most upvoted comments

Shit @tienifr , thanks for the call out (we should have an automate post in the future stating the assign and merge dates to prevent this from happening) @tienifr hired on Feb 7 Merged on Feb 10 Eligible for bonus 🎉

Both @tienifr and @thesahindia have been bonused $500 for timeliness. Thanks for the reminder @tienifr

@thesahindia @jasperhuangg

Regression Test Proposal

Bug: Link syntax doesn’t work on link ended with question mark

Proposed Test Steps:

  1. Open the app
  2. Open a chat and send: [link example](http://example.com/?)
  3. Verify that it displays the example.com link and when we hover over the link, the URL http://example.com/? is displayed

Do we 👍 or 👎