[HOLD for payment 2024-06-20] [Search v1] Improve performance of ReportListItem

[luacmartins]

Coming from this PR, let's apply these performance improvements to ReportListItem.

Additionally, we should also address these comments:

• [No QA][Search v1] Create ReportListItem #42410 (comment)
• [No QA][Search v1] Create ReportListItem #42410 (comment)
• [No QA][Search v1] Create ReportListItem #42410 (comment)
• [No QA][Search v1] Create ReportListItem #42410 (comment)
• [No QA][Search v1] Create ReportListItem #42410 (comment)
• [No QA][Search v1] Create ReportListItem #42410 (comment)

cc @WojtekBoman

Issue Owner

Current Issue Owner: @alexpensify

[melvin-bot]

Triggered auto assignment to @alexpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

[DylanDylann]

@luacmartins Could I take over this issue as C+ contributor to conveniently follow updates there?

[alexpensify]

I believe that this one has a few PRs in the works, but there was one moved to production yesterday. For now, adding the Reviewing label.

[luacmartins]

We haven't started work on this one yet. @WojtekBoman can you comment here so I can assign this to you please?

[WojtekBoman]

Hi! I'm going to work on that :)

[alexpensify]

@luacmartins - one of the PR has merged, are we waiting for more actions here? Thanks!

[luacmartins]

No, I think we're good for now just need to get this deployed.

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

• [Search v1] Improve performance of ReportListItem #43092

If no regressions arise, payment will be issued on 2024-06-18. 🎊

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

• @WojtekBoman does not require payment (Contractor)
• @DylanDylann requires payment (Needs manual offer from BZ)

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

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

[melvin-bot]

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

• [Search v1] Improve performance of ReportListItem #43092

If no regressions arise, payment will be issued on 2024-06-20. 🎊

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

• @WojtekBoman does not require payment (Contractor)
• @DylanDylann requires payment (Needs manual offer from BZ)

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

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

[DylanDylann]

No need payment here. The PR is merged without my review

[luacmartins]

We're good to close this one then!

[alexpensify]

Keeping closed and there is no Upwork job. No action on this payment date per this note:

#42720 (comment)