[$500] Chat – New message button appears when create IOU

[lanitochka17]

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.56-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4444284
Email or phone of affected tester (no customers): applausetester+jp_e_category@applause.expensifail.com
Issue reported by: Applause - Internal Team

Action Performed:

1. Go to https://staging.new.expensify.com/
2. Log in
3. Navigate to account with a long conversation history
4. Send a message
5. Create IOU

Expected Result:

New message button not appears on the top of the page when create IOU

Actual Result:

New message button appears on the top of the page when create IOU

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

Bug6423672_1711138394644.IOU.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0133b550ad31059d52
• Upwork Job ID: 1772371228780400640
• Last Price Increase: 2024-04-01

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[lanitochka17]

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

[melvin-bot]

Triggered auto assignment to @yuwenmemon (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[gijoe0295]

Same RCA as #38830. sortedVisibleReportActions contains only visible actions while sortedReportActions contains invisible as well. For hasNewestReportAction check below, we should use the visible one:

App/src/pages/home/report/ReportActionsList.tsx

Line 191 in 665a8ad

const hasNewestReportAction = sortedReportActions?.[0].created === report.lastVisibleActionCreated;

[gijoe0295]

Investigate more and I'm not sure we can consolidate this with #38830. My RCA above is correct but I found no solution yet because I think using sortedReportActions is on purpose. Let's treat this separately.

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[Pujan92]

I am not sure but RequestMoney API onyx response for the report seems sending old time for lastVisibleActionCreated which could be the issue in below check.

App/src/pages/home/report/ReportActionsList.tsx

Line 191 in 665a8ad

const hasNewestReportAction = sortedReportActions?.[0].created === report.lastVisibleActionCreated;

Screen.Recording.2024-03-23.at.16.37.45.mov

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[yuwenmemon]

Unable to reproduce on the latest main