Skip to content

[Due for payment 2026-08-05] [Split #94595] Batch H - Remove useOnyx deps from Nav/Wallet/Travel/Debug #95581

Description

@mountiny

Sub-issue of #94595.
Source plan: fabioh8010 comment.

Goal

Remove the deprecated 3rd dependencies argument from Navigation, Wallet, Travel, and Debug components.

Scope (8 files)

  • Navigation/NavigationTabBar/InboxTabButton
  • Navigation/QuickCreationActionsBar/index
  • inbox/sidebar/FABPopoverContent/menuItems/CreateReportMenuItem (USECALLBACK)
  • settings/Wallet/ExpensifyCardPage/index
  • Travel/TripDetailsPage (RESTRUCTURE)
  • Debug/Report/DebugReportActions
  • Debug/Report/DebugReportPage
  • Debug/ReportAction/DebugReportActionPage

Change types

RESTRUCTURE (TripDetailsPage); USECALLBACK (CreateReportMenuItem).

Dependencies

Blocked by PR 1 (ESLint guard) merged.

Tests / QA

Bottom-tab Inbox button (unread state), FAB create-report menu, Expensify Card page, Trip details (RESTRUCTURE — verify traveler details), Debug report/report-action pages.

Acceptance criteria

  1. npm run fmt on changed files
  2. npm run lint-changed — seatbelt count for rulesdir/no-useonyx-dependencies-arg must drop (never increase)
  3. npm run typecheck stays green
  4. npm run react-compiler-compliance-check check <files> — USECALLBACK/bailout files must not gain new failures vs main
  5. Relevant npm run test for touched areas
  6. Manual smoke test per batch QA steps — no stale data, infinite re-renders, or JS console errors
Issue OwnerCurrent Issue Owner: @hoangzinh

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions