search

safe-global / safe-cli

Command line utility for Safe
MIT License
208 stars 68 forks source link

Fix hw_wallet overwrites previous signatures #411

Closed moisses89 closed 3 months ago

moisses89 commented 3 months ago

Description

sign_safe_tx of HwWalletManager was removing previous signatures in the SafeTx.

Solution

Concatenate previous signatures and export the sorted signatures.

coveralls commented 3 months ago

Coverage Status

coverage: 87.136% (-0.02%) from 87.16% when pulling 78bec1e15250ea7c412244f119783657dc7df746 on fix_overwrite_signature into cc5b9f1c080a2e7bbaca831bd41b699c89f946f3 on main.