[HOLD for payment 2024-08-09] LHN - Long pressing chat does not show "Copy Onx Data" on all platforms

[m-natarajan]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Found when validating ##45079
Version Number: 9.0.7-4
Reproducible in staging?: y
Reproducible in production?: no, new feature
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: applause internal team
Slack conversation:

Action Performed:

1. Access New Expensify app
2. Sign into a valid account (Beta enabled)
3. Long press on LHN

Expected Result:

User expects to see the option "Copy Onx Data" in popup menu

Actual Result:

User can not see this option on iOS and Android app

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

View all open jobs on GitHub

Issue Owner

Current Issue Owner: @trjExpensify

[melvin-bot]

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

[melvin-bot]

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

[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.

[trjExpensify]

From the PR, this is behind a beta and dev/staging only, so I don't think it's a Web blocker, or even really a App blocker.

Adds a new field for the context menu that allows to quickly copy onyx data of any report from the LHP for staging, dev and internal build environments.

CC: @puneetlath @sobitneupane @cdOut @war-in

[cdOut]

This is not a blocker as it does not affect production and is only enabled for staging / dev and internal builds.

[cdOut]

I'll look into it since I worked on the original PR implementing this feature, however as far as I remember it did work when I tested it on all platforms.

@m-natarajan did this happen on staging only? It seems like the conditional for checking the environment might have failed for some case there.

[trjExpensify]

@m-natarajan did this happen on staging only? It seems like the conditional for checking the environment might have failed for some case there.

@m-natarajan bump on this Q!

[cdOut]

Second on the bump, I tried reproducing this one but I wasn't able to, were you testing this one on the emulators or using a physical device?

[m-natarajan]

@trjExpensify @cdOut Yes on staging only, I am using physical device (iPhone Xs, App version 9.0.11-0) and able to reproduce.

[cdOut]

Could you see whether it also happens for you when running on emulators?

[m-natarajan]

@cdOut Applause QA never uses emulators.

[cdOut]

Alright, I'll test it on a physical device and see if I can replicate it then. Thank you for clearing that up!

[cdOut]

@m-natarajan has this been tested using an internal build or a staging build?

[trjExpensify]

From the screenie in the OP, it's a staging build:

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.15-9 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• LHN - Long pressing chat does not show "Copy Onx Data" on all platforms #46482

If no regressions arise, payment will be issued on 2024-08-09. 🎊

For reference, here are some details about the assignees on this issue:

• @cdOut does not require payment (Contractor)

[melvin-bot]

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [@bondydaa] The PR that introduced the bug has been identified. Link to the PR:
• [@bondydaa] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [@bondydaa] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [@cdOut] Determine if we should create a regression test for this bug.
• [@cdOut] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [@trjExpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[trjExpensify]

@sobitneupane needs paying here for the C+ review.

Payment summary as follows:

• $250 for the C+ review

Go ahead and request!

[garrettmknight]

$250 approved for @sobitneupane