Skip to content

Commit 912814d

Browse files
committed
address review nits
1 parent b63917a commit 912814d

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

src/components/MoneyRequestConfirmationListFooter/hooks/useFooterDerivedFlags.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@ function useFooterDerivedFlags({
9999
const shouldShowBillable = policy?.disabledFields?.defaultBillable === false;
100100
const shouldShowReimbursable =
101101
(isPolicyExpenseChat || isTrackExpense) && !!policy && policy?.disabledFields?.reimbursable !== true && !isManagedCardTransaction(transaction) && !isTypeInvoice;
102-
103102
const shouldNavigateToUpgradePath = !policyForMovingExpensesID && !shouldSelectPolicy;
104103
const shouldShowTimeRequestFields = isTimeRequest && action === CONST.IOU.ACTION.CREATE;
105104

src/components/MoneyRequestConfirmationListFooter/hooks/useReceiptThumbnailSource.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ function useReceiptThumbnailSource({transaction, receiptPath, receiptFilename}:
4343
// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing -- empty string fallback is intentional: we want to skip the empty-string case in the OR chain, not treat it as a valid source
4444
setInitialLocalSource({source: thumbnailUri || resolvedReceiptImageStr || '', resolvedImage: resolvedReceiptImageStr});
4545
}
46+
4647
// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing -- empty string fallback is intentional: we want to skip the empty-string case in the OR chain, not treat it as a valid source
4748
const effectiveReceiptSource = isLocalFile ? initialLocalSource.source || '' : resolvedThumbnail || resolvedReceiptImage || '';
4849

0 commit comments

Comments
 (0)