Skip to content

Commit

Permalink
Merge pull request #51076 from Expensify/lucien/fix-saveSearch-tooltip
Browse files Browse the repository at this point in the history
[Search] [Saved Searches] Do not render tooltip more than once
  • Loading branch information
pecanoro authored Oct 22, 2024
2 parents 245be22 + f6ec009 commit 7654693
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/pages/Search/AdvancedSearchFilters.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ import ONYXKEYS from '@src/ONYXKEYS';
import ROUTES from '@src/ROUTES';
import type {SearchAdvancedFiltersForm} from '@src/types/form';
import type {CardList, PersonalDetailsList, Report} from '@src/types/onyx';
import {isEmptyObject} from '@src/types/utils/EmptyObject';

const baseFilterConfig = {
date: {
Expand Down Expand Up @@ -287,7 +286,8 @@ function AdvancedSearchFilters() {
return;
}

if (isEmptyObject(savedSearches)) {
// We only want to show the tooltip once, the NVP will not be set if the user has not saved a search yet
if (!savedSearches) {
SearchActions.showSavedSearchRenameTooltip();
}

Expand Down

0 comments on commit 7654693

Please sign in to comment.