Skip to content

Migrate useSearchSelector.base.ts from useOptionsList to usePersonalDetailOptions (part 6)#96440

Open
shubham1206agra wants to merge 2 commits into
Expensify:mainfrom
shubham1206agra:refactor-useSelector-6
Open

Migrate useSearchSelector.base.ts from useOptionsList to usePersonalDetailOptions (part 6)#96440
shubham1206agra wants to merge 2 commits into
Expensify:mainfrom
shubham1206agra:refactor-useSelector-6

Addressed comments

44ab033
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 18, 2026 in 0s

48.10% (-0.01%) compared to 6b62520

View this Pull Request on Codecov

48.10% (-0.01%) compared to 6b62520

Details

Codecov Report

❌ Patch coverage is 40.00000% with 24 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...pages/iou/request/MoneyRequestAttendeeSelector.tsx 0.00% 23 Missing ⚠️
src/libs/PersonalDetailOptionsListUtils/index.ts 93.33% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/hooks/usePersonalDetailSearchSelector/base.ts 95.89% <100.00%> (+0.11%) ⬆️
src/libs/OptionsListUtils/index.ts 86.53% <ø> (-0.37%) ⬇️
...pages/iou/request/step/IOURequestStepAttendees.tsx 0.00% <ø> (ø)
src/libs/PersonalDetailOptionsListUtils/index.ts 72.22% <93.33%> (+1.57%) ⬆️
...pages/iou/request/MoneyRequestAttendeeSelector.tsx 0.00% <0.00%> (ø)

... and 11 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.