Salvatore Ingala
Salvatore Ingala
Hello! First of all, great work! And thanks @mflaxman for checking out the new Ledger app, and making a POC. I'd really hope to get integrations in the ecosystem, and...
> Although I'd like Sparrow to support Ledger's full multisig functionality, it's not particularly practical to support different applications for the large and growing number of hardware wallets. I'd like...
I realized I didn't fully answer your question: > although I haven't fully understood what kind of wallets are _not_ supported. The goal is to support _all_ sane miniscript wallets....
I tried to do some work in the [214-bootstrap-signal](/bigspider/python-teos/tree/214-bootstrap-signal) branch; it's easy to do it using an Event for processes that we create with `multiprocessing.Process`, but I didn't manage to...
After #188, this should be done, but not for e2e tests that still expect to be run in sequence.
> I feel that, in order to avoid this kind of problems, the appointment should acknowledge wether the transaction is standard or not, in the same way it should acknowledge...
(of course this does not apply to double-spend attacks mentioned above, we should make sure to consider those)
@achow101, I didn't work on the tests − I'm not yet familiar with the test suite. Please advise accordingly if there are improvements you'd like me to make.
I managed to run the test suite locally, but it seems to run with an old binary taken from the speculos repository. Not sure how to fix it. Meanwhile, I...
@achow101 I disabled again in 00a8c1b32e02d80fb72dc4fbfaae44b37474ece6 the tests for multisig address display, since I'm not too sure how best to adapt the existing tests. Maybe we could leave that to...