@ticketboothapp/booking 1.2.173 → 1.2.174

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ticketboothapp/booking",
-  "version": "1.2.173",
+  "version": "1.2.174",
   "private": false,
   "sideEffects": [
     "**/*.css",

package/src/components/booking/AdminChangeBookingFlow.tsx

@@ -833,12 +833,6 @@ export function AdminChangeBookingFlow({
     latestChangeQuote?.allowedRefundDispositionsByOperationId,
     latestChangeQuote?.removedValueByOperationId,
   ]);
-  useEffect(() => {
-    if (!isProviderDashboardChange || responseRefundDecisionOperations.length === 0) return;
-    setRefundDispositionsByOperationId((current) =>
-      defaultPendingRefundDispositions(responseRefundDecisionOperations, current),
-    );
-  }, [isProviderDashboardChange, responseRefundDecisionOperations]);
   const activeRefundDecisionContext = useMemo(() => mergeAdminRefundDecisionContext({
     current: refundDecisionContext,
     selectionKey: refundDecisionSelectionKey,
@@ -857,9 +851,19 @@ export function AdminChangeBookingFlow({
       ? responseReturnPriceTreatmentOperations
       : [];
   const activeRefundDispositionsByOperationId = useMemo(() => {
+    // Provider changes default discovered refund operations to pending at
+    // settlement time. Do not write those response-derived defaults into the
+    // explicit input state: doing so changes the request key after the first
+    // quote and immediately triggers an identical second quote.
+    const dispositionsWithPendingRefundDefaults = isProviderDashboardChange
+      ? defaultPendingRefundDispositions(
+          activeRefundDecisionContext?.operations ?? [],
+          refundDispositionsByOperationId,
+        )
+      : refundDispositionsByOperationId;
     const dispositionsWithReturnPriceDefaults = defaultKeepOriginalReturnPriceDispositions(
       activeReturnPriceTreatmentOperations,
-      refundDispositionsByOperationId,
+      dispositionsWithPendingRefundDefaults,
     );
     return selectRefundDispositionsForOperations(
       activeRefundDecisionContext,
@@ -869,6 +873,7 @@ export function AdminChangeBookingFlow({
   }, [
     activeRefundDecisionContext?.operations,
     activeReturnPriceTreatmentOperations,
+    isProviderDashboardChange,
     refundDispositionsByOperationId,
   ]);
   const handleRefundDispositionChange = useCallback((
@@ -910,7 +915,9 @@ export function AdminChangeBookingFlow({
     adminReverseAdjustmentIds,
     needsAdjustmentDiscovery:
       isProviderDashboardChange && !hasHydratedExistingAdjustments,
-    refundDispositionsByOperationId: activeRefundDispositionsByOperationId,
+    // Only explicit user inputs belong to quote identity. Automatic provider
+    // defaults are derived separately for settlement/submission below.
+    refundDispositionsByOperationId,
     useAdminFeAuthoritativeQuote,
     latestChangeQuote,
     setLatestChangeQuote,