[HOLD for payment 2024-10-14] [HOLD for payment 2024-10-11] [$250] [Search v2.4] [App] Add Recent Searches and Recent Chats sections

[luacmartins]

Implement this section of the doc

cc @Kicu @289Adam289 @Guccio163 @SzymczakJ

Issue Owner

Current Issue Owner: @

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021834650319760395608
• Upwork Job ID: 1834650319760395608
• Last Price Increase: 2024-09-13

Issue Owner

Current Issue Owner: @Christinadobrzyn

[melvin-bot]

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

[SzymczakJ]

Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and I’d like to work on this issue!

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021834650319760395608

[melvin-bot]

Current assignee @rayane-djouah is eligible for the External assigner, not assigning anyone new.

[melvin-bot]

@luacmartins, @Christinadobrzyn, @rayane-djouah, @SzymczakJ Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[Christinadobrzyn]

hi @SzymczakJ do you have an update on this for the team? TY!

[SzymczakJ]

The draft PR should be ready tomorrow.

[melvin-bot]

@luacmartins, @Christinadobrzyn, @rayane-djouah, @SzymczakJ Whoops! This issue is 2 days overdue. Let's get this updated quick!

[luacmartins]

Draft PR up

[Christinadobrzyn]

Monitoring - #49457

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

⚠️ 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.

[Christinadobrzyn]

Hey @rayane-djouah and @SzymczakJ can you check these ^ deploy blocker messages to see if they apply here?

[Kicu]

but @jasperhuangg removed the deploy blocker labels from the issue as they were not able to reproduce it.
So I don't think there is anything to verify here if there is no clear repro.

[luacmartins]

Those blockers apply here, but they were already fixed with this PR/CP, that's why the blocker label was removed

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

• Add recent serches and recent chats in Search router #49457
• Fix performance of SearchRouter #50143

If no regressions arise, payment will be issued on 2024-10-11. 🎊

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

• @rayane-djouah requires payment (Needs manual offer from BZ)
• @SzymczakJ 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:

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

[rayane-djouah]

Regression tests will be handled as part of the project wrap-up

[melvin-bot]

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

• Fix performance of SearchRouter #50143

If no regressions arise, payment will be issued on 2024-10-14. 🎊

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

• @rayane-djouah requires payment (Needs manual offer from BZ)
• @SzymczakJ 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:

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

[rayane-djouah]

Regression tests will be handled as part of the project wrap-up

[Christinadobrzyn]

Payouts due:

• Reviewer: $250 @rayane-djouah (Upwork offer - https://www.upwork.com/nx/wm/offer/104402695)

[rayane-djouah]

@Christinadobrzyn - Offer accepted, thank you!

[melvin-bot]

Payment Summary

Upwork Job

• ROLE: @rayane-djouah payment summary here - [HOLD for payment 2024-10-14] [HOLD for payment 2024-10-11] [$250] [Search v2.4] [App] Add Recent Searches and Recent Chats sections #49116 (comment)
• Contributor: @SzymczakJ is from an agency-contributor and not due payment

BugZero Checklist (@Christinadobrzyn)

• I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
• I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants/1834650319760395608/hired)
• I have paid out the Upwork contracts or cancelled the ones that are incorrect
• I have verified the payment summary above is correct

[Christinadobrzyn]

It's payday - payment summary here - #49116 (comment)

No regression test so closing this out.