Skip to content

Commit

Permalink
Merge pull request #56540 from FitseTLT/fix-to-show-loading-on-submit…
Browse files Browse the repository at this point in the history
…-button

Fix - show loading on submit button after expense creation confirmed
  • Loading branch information
arosiclair authored Feb 11, 2025
2 parents 08856a8 + 294962f commit 5b05237
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/components/MoneyRequestConfirmationList.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -954,6 +954,7 @@ function MoneyRequestConfirmationList({
}}
enterKeyEventListenerPriority={1}
useKeyboardShortcuts
isLoading={isConfirmed}
/>
) : (
<ButtonWithDropdownMenu
Expand All @@ -962,6 +963,7 @@ function MoneyRequestConfirmationList({
options={splitOrRequestOptions}
buttonSize={CONST.DROPDOWN_BUTTON_SIZE.LARGE}
enterKeyEventListenerPriority={1}
isLoading={isConfirmed}
useKeyboardShortcuts
/>
);
Expand All @@ -979,7 +981,7 @@ function MoneyRequestConfirmationList({
{button}
</>
);
}, [isReadOnly, iouType, confirm, bankAccountRoute, iouCurrencyCode, policyID, splitOrRequestOptions, styles.ph1, styles.mb2, errorMessage]);
}, [isReadOnly, iouType, confirm, isConfirmed, bankAccountRoute, iouCurrencyCode, policyID, splitOrRequestOptions, styles.ph1, styles.mb2, errorMessage]);

const listFooterContent = (
<MoneyRequestConfirmationListFooter
Expand Down

0 comments on commit 5b05237

Please sign in to comment.