App: [HOLD ON #35977] [$500] [MEDIUM] `Attachment` shown in the LHN when sending a message with image

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: 1.4.13-14 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 Expensify/Expensify Issue URL: Issue reported by: @quinthar Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1708811957655339

Action Performed:

Open any chat
Send a message with attachment
Look at the LHN as a recipient

Expected Result:

In the LHN first line of the message should display

Actual Result:

Attachment is displayed

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

https://github.com/Expensify/App/assets/38435837/45c13785-4037-41d3-addd-7d17341bab4a image (5)

View all open jobs on GitHub

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a1ad1258cf6daa29
Upwork Job ID: 1761910899454898176
Last Price Increase: 2024-03-18

About this issue

Original URL
State: open
Created 4 months ago
Comments: 37 (24 by maintainers)

Most upvoted comments

Also agree with the proposal to hold this

kevinksullivan on Mar 19, 2024

@brandonhenry You might be right that the FE already implemented the send attachment with the message, but still if the reportActions are created separately, then the last text shown in the LHN is from the last reportActions.

@dukenv0307 Your proposal seems to have the right path, but we have ongoing discussions about implementing the inline image here https://expensify.slack.com/archives/C01GTK53T8Q/p1707270017261459

@tgolen Do you mind confirming if this needs to hold for https://github.com/Expensify/App/issues/35977 or https://github.com/Expensify/App/issues/37246?

mollfpr on Feb 29, 2024