[Search v2.3] - Educational tooltip appears repeatedly after relogin

[IuliiaHerets]

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: 9.0.35-0 Reproducible in staging?: Y Reproducible in production?: N/A - new feature, doesn't exist in prod Issue was found when executing this PR: https://github.com/Expensify/App/pull/48566 Email or phone of affected tester (no customers): applausetester+kh010901@applause.expensifail.com Issue reported by: Applause Internal Team

Action Performed:

1. Go to staging.new.expensify.com
2. Go to Search.
3. Click Filters.
4. Add a few filters.
5. Click Save search.
6. Repeat Step 3 to 5 a few times to create more saved searches.
7. Go to Troubleshoot > Clear cache and restart > Reset and refresh (or log out and log in again).
8. Go to Search.

Expected Result:

Educational tooltip will not appear again as it has appeared previously.

Actual Result:

Educational tooltip appears repeatedly for each saved search after relogin.

Workaround:

Unknown

Platforms:

• [ ] Android: Native
• [ ] Android: mWeb Chrome
• [ ] iOS: Native
• [ ] iOS: mWeb Safari
• [x] MacOS: Chrome / Safari
• [x] MacOS: Desktop

Screenshots/Videos

https://github.com/user-attachments/assets/3f7d9f4c-5722-428e-a480-9ba7306d421d

View all open jobs on GitHub

Issue Owner

Current Issue Owner: @lakchote

[github-actions[bot]]

:wave: 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.

[melvin-bot[bot]]

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

[melvin-bot[bot]]

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

[IuliiaHerets]

We think that this bug might be related to #wave-control

[gijoe0295]

Edited by proposal-police: This proposal was edited at 2023-10-04T16:20:00Z.

Proposal

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

1. Tooltips show for every saved search items
2. Tooltips reappear after log in again

What is the root cause of that problem?

1. We show tooltip for every items. As long as shouldRenderTooltip is true, it's true for every items:

https://github.com/Expensify/App/blob/2dc08e931bb58a72ab4d09e74165fe9be0c002a7/src/pages/Search/SearchTypeMenu.tsx#L128

2. NVP_SHOULD_HIDE_SAVED_SEARCH_RENAME_TOOLTIP is a local value and gets clear out once signing out.

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

1. Add index param to createSavedSearchMenuItem and only show tooltip for the first item:

https://github.com/Expensify/App/blob/2dc08e931bb58a72ab4d09e74165fe9be0c002a7/src/pages/Search/SearchTypeMenu.tsx#L161

https://github.com/Expensify/App/blob/2dc08e931bb58a72ab4d09e74165fe9be0c002a7/src/pages/Search/SearchTypeMenu.tsx#L99

shouldRenderTooltip: index === 0 && !shouldHideSavedSearchRenameTooltip

2. Add an API endpoint to modify NVP_SHOULD_HIDE_SAVED_SEARCH_RENAME_TOOLTIP just like we did here:

https://github.com/Expensify/App/blob/e1d133b5af033de731a0fbeaf1e1b9d83243adc1/src/libs/actions/User.ts#L1288

[luacmartins]

This is known and NAB for now. We're addressing it as a follow up.

[lakchote]

Also to be solved:

The tooltip should not show over a dialog.

Steps:

1. Clicked on an expense on the search result
2. Clicked on the receipt thumbnail
3. The tooltip is still showing above the dialog

Slack discussion