Skip to content

Commit 39610da

Browse files
authored
Merge pull request Expensify#66631 from margelo/@perunt/clean
[No QA] Remove warning
2 parents 791339b + 165c068 commit 39610da

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/components/Search/index.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ type SearchProps = {
6060
queryJSON: SearchQueryJSON;
6161
onSearchListScroll?: (event: NativeSyntheticEvent<NativeScrollEvent>) => void;
6262
contentContainerStyle?: ContentStyle;
63-
// contentContainerStyle?: StyleProp<ViewStyle>; // TODO: remove this
6463
searchResults?: SearchResults;
6564
handleSearch: (value: SearchParams) => void;
6665
isMobileSelectionModeEnabled: boolean;
@@ -243,6 +242,8 @@ function Search({queryJSON, searchResults, onSearchListScroll, contentContainerS
243242

244243
handleSearch({queryJSON, offset});
245244
// We don't need to run the effect on change of isFocused.
245+
// eslint-disable-next-line react-compiler/react-compiler
246+
// eslint-disable-next-line react-hooks/exhaustive-deps
246247
}, [handleSearch, isOffline, offset, queryJSON]);
247248

248249
useEffect(() => {
@@ -543,7 +544,7 @@ function Search({queryJSON, searchResults, onSearchListScroll, contentContainerS
543544
return;
544545
}
545546
setOffset(offset + CONST.SEARCH.RESULTS_PAGE_SIZE);
546-
}, [offset, searchResults?.search?.hasMoreResults, shouldShowLoadingMoreItems, shouldShowLoadingState]);
547+
}, [isFocused, offset, searchResults?.search?.hasMoreResults, shouldShowLoadingMoreItems, shouldShowLoadingState]);
547548

548549
const toggleAllTransactions = useCallback(() => {
549550
const areItemsGrouped = !!groupBy;

0 commit comments

Comments
 (0)