Merge pull request Expensify#54986 from nkdengineer/fix/54725

fix: Pay userName emit no sound
software-mansion-labs · Jan 14, 2025 · e9fc8b3 · e9fc8b3
2 parents 8d66dab + 102a487
commit e9fc8b3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/components/MoneyRequestConfirmationList.tsx b/src/components/MoneyRequestConfirmationList.tsx
@@ -814,6 +814,9 @@ function MoneyRequestConfirmationList({
                     return;
                 }
                 Log.info(`[IOU] Sending money via: ${paymentMethod}`);
+                if (shouldPlaySound) {
+                    playSound(SOUNDS.DONE);
+                }
                 onSendMoney?.(paymentMethod);
             }
         },

diff --git a/src/pages/iou/request/step/IOURequestStepConfirmation.tsx b/src/pages/iou/request/step/IOURequestStepConfirmation.tsx
@@ -673,7 +673,7 @@ function IOURequestStepConfirmation({
                     shouldShowSmartScanFields={isMovingTransactionFromTrackExpense ? transaction?.amount !== 0 : requestType !== CONST.IOU.REQUEST_TYPE.SCAN}
                     action={action}
                     payeePersonalDetails={payeePersonalDetails}
-                    shouldPlaySound={false}
+                    shouldPlaySound={iouType === CONST.IOU.TYPE.PAY}
                     isConfirmed={isConfirmed}
                 />
             </View>