[HOLD for payment 2024-07-24] Add copy report onyx object to clipboard function on dev/staging

[puneetlath]

In order to make troubleshooting easier, let's add a new option to the right-click/long-press menu on reports in the LHN. The new option will be called Copy Onyx data and it will only be visible on dev/staging.

Clicking on it will copy the Onyx data for the report object onto the user's clipboard. Without opening the report.

Issue Owner

Current Issue Owner: @puneetlath

[melvin-bot]

Current assignee @puneetlath is eligible for the NewFeature assigner, not assigning anyone new.

[ShridharGoel]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Add copy report onyx object to clipboard function on dev/staging

What is the root cause of that problem?

New feature.

What changes do you think we should make in order to solve the problem?

Add a new option to ContextMenuActions.tsx.

Logic on pressing the option:

const report = ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportID}`];
Clipboard.setString(JSON.stringify(report));

shouldShow should be true when type === CONST.CONTEXT_MENU_TYPES.REPORT and env is dev/staging.

{
    isAnonymousAction: false,
    textTranslateKey: 'reportActionContextMenu.copyOnyxData',
    icon: Expensicons.Copy,
    successIcon: Expensicons.Checkmark,
    shouldShow: (type, reportAction, isArchivedRoom, betas, menuTarget, isChronosReport, reportID, isPinnedChat, isUnreadChat) =>
        type === CONST.CONTEXT_MENU_TYPES.REPORT && isDevelopment(), // Can add staging check also.
    onPress: (closePopover, {reportID}) => {
        const report = ReportConnection.getAllReports()?.[`${ONYXKEYS.COLLECTION.REPORT}${reportID}`];
        Clipboard.setString(JSON.stringify(report));
    },
    getDescription: () => {},
},

Screen.Recording.2024-07-09.at.12.02.55.AM.mov

[TMisiukiewicz]

I'm Tomasz from Callstack and I'd like to work on this issue

[ShridharGoel]

@puneetlath It's a simple change, can it be made external? I've added a proposal above.

[cdOut]

Hi, I'm Tymoteusz Kałuzieński from Software Mansion, the expert agency, and I'll be working on this issue.

[puneetlath]

@ShridharGoel I'd like to get it out fast, so working with an agency contributor on it directly. That's why I didn't mark it external. Thanks for your interest though!

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

[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.7-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Add copy report onyx object to clipboard function on dev/staging #45079

If no regressions arise, payment will be issued on 2024-07-24. 🎊

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

• @cdOut does not require payment (Contractor)

[melvin-bot]

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

• [@cdOut] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
• [@puneetlath] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

[melvin-bot]

Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO.

[puneetlath]

I believe @sobitneupane needs to be paid here. Right @sobitneupane?

[sobitneupane]

Yes @puneetlath. I reviewed the PR linked with this issue.

[puneetlath]

Payment summary:

• $250 - @sobitneupane - C+

Please request on NewDot @sobitneupane

[JmillsExpensify]

$250 approved for @sobitneupane