Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.26k stars 2.69k forks source link

mWeb - Search - Search page shows attachment as html code #45215

Closed lanitochka17 closed 1 month ago

lanitochka17 commented 1 month ago

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.4 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to https://staging.new.expensify.com/home
  2. Open any 1:1 DM
  3. Enter a text & tap plus icon and add an attachment
  4. Send the message
  5. Tap app back button
  6. Tap search icon

Expected Result:

Search page must not show attachment as html code

Actual Result:

Search page shows attachment as html code

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/b9acc882-9b52-4b53-9605-367e2313a926

View all open jobs on GitHub

melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @johncschuster (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.

lanitochka17 commented 1 month ago

@johncschuster FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

lanitochka17 commented 1 month ago

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

bernhardoj commented 1 month ago

BE issue, the lastMessageText contains the <br /> tag.

image
johncschuster commented 1 month ago

Taking it internal

johncschuster commented 1 month ago

Ah, it looks like I placed this in the wrong project. Updated to #wave-collect.

johncschuster commented 1 month ago

Raising in Slack

melvin-bot[bot] commented 1 month ago

@johncschuster Whoops! This issue is 2 days overdue. Let's get this updated quick!

johncschuster commented 1 month ago

Chatted with Tom, and he suggested I bring this to #vip-vsb. Checking the priority there.

johncschuster commented 1 month ago

Closing this for now