Repository navigation

Decouple the file download modal from report and Expensify Card logic - #104534

Open

MelvinBot wants to merge 9 commits into

mainfrom

claude-decoupleFileDownloadModal

Open

Decouple the file download modal from report and Expensify Card logic#104534
MelvinBot wants to merge 9 commits into
mainfrom
claude-decoupleFileDownloadModal

Commits on Oct 21, 2026
- Decouple the file download modal from report and Expensify Card logic
  
  MelvinBot
  and
  daledah
  committed
  8bfeb94
- Fix: remove unused FileDownloadStatus type export flagged by knip
  
  MelvinBot
  and
  daledah
  committed
  152d7d6
- Move the file download modal lifecycle into a generic useFileDownloadModal hook
  
  MelvinBot
  and
  daledah
  committed
  ee24686
Commits on Oct 22, 2026
Commits on Oct 23, 2026
- Split the report PDF modal comment into two sentences
  
  MelvinBot
  and
  daledah
  committed
  a66607c
- Move the PDF download modal subscriptions into components the global modal stack renders
  
  MelvinBot
  and
  daledah
  committed
  7e2a88c
- Wrap openPDFDownload, revert the closeModalByID change, and keep resolveModal out of FileDownloadModal
  
  MelvinBot
  and
  daledah
  committed
  4895720