Skip to content

Commit 16936af

Browse files
fix(shielded): log only a completed wallet drain as a succeeded proof
A pending drain was sent but not confirmed, which the other shielded routes report by throwing and so log as succeeded=false. Logging it as succeeded mixed unconfirmed drains into the timing comparison. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 9957ce7 commit 16936af

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎DashWallet/Sources/UI/Payments/InternalTransfer/Shielded/ShieldedTransferCoordinator.swift‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -580,7 +580,7 @@ final class ShieldedTransferCoordinator: ObservableObject {
580580
await env.manager.shieldedFundFromWalletDrain(
581581
walletId: env.walletId, operationId: operation.id,
582582
recipientRaw43: Self.drainRecipientRaw43(operation.recipient, ownAddress: env.shieldedRecipient))
583-
}, succeeded: { $0.drainVerdict != .notSent })
583+
}, succeeded: { $0.drainVerdict == .completed })
584584
if let outpoint = result.outpoint {
585585
lastAssetLockOutPoint = (outpoint.txidWire, outpoint.vout)
586586
}

0 commit comments

Comments
 (0)