Skip to content

Commit

Permalink
fix: confirmations required for signing
Browse files Browse the repository at this point in the history
  • Loading branch information
dtdang committed Oct 25, 2023
1 parent af65c12 commit 254bfea
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ape_safe/_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ def pending(cli_ctx, network, sign_with_local_signers, execute, alias):
)

# Add signatures, if was requested to do so.
if sign_with_local_signers and len(confirmations) < safe.confirmations_required:
if sign_with_local_signers and len(confirmations) < safe.confirmations_required - 1:
safe.add_signatures(safe_tx, confirmations)
cli_ctx.logger.success(f"Signature added to 'Transaction {safe_tx.nonce}'.")

Expand Down

0 comments on commit 254bfea

Please sign in to comment.