697fb16db976d0bc643381b10342f2df650a4386 Add more information and checkbox to register descriptor step (edouard)
Pull request description:
close#297
ACKs for top commit:
darosior:
utACK 697fb16db976d0bc643381b10342f2df650a4386 -- text looks good to me.
Tree-SHA512: 2bfdad9e512612e8cdc5247d0d2a4b35686f97b31fea508f6674baa5e7b1beb5a7929377a9cb3b843644597d186bff67f0200c1d08b6bd2d335477dbcaa83456
263f00150d37908d97bb491cd0dbc2180f36b654 fix: spend minimal amount in btc (edouard)
Pull request description:
"> 5000 sats" message was confusing with the actual amount in btc requirement
ACKs for top commit:
edouardparis:
Self-ACK 263f00150d37908d97bb491cd0dbc2180f36b654
Tree-SHA512: 7846344af9e044db5431d583214c06cdbabfbb5a553be975531e76b5218bec3c7fd0467d11fe4fa2845226516cf09e45d0948df17b2a8c723a3cedb212fe34d0
ea4f02172824371ef162db9ee0b974ef4597a541 fix hws list reloading (edouard)
Pull request description:
I wanted the refresh button to add more harware wallet to the list without erasing the previous list
in order to not cut the existing communication channels.
But with the new display of "unsupported wallet" this behaviour can corrupt the list. An "unsupported wallet" can become "supported" because user switch app or unlock.
It is preferable to reset the full list and drop the objects after a refresh.
ACKs for top commit:
edouardparis:
Self-ACK ea4f02172824371ef162db9ee0b974ef4597a541
Tree-SHA512: 530cf6e82f5bfd325e45c0eb19af94a58e930ecc15a692e0d545ceec2105122ed539a9f671449d2bc05e87c0ba562a1f192076a5cd1a2264f322b5a9094c1d9a
I wanted the refresh button to add more harware wallet
to the list without erasing the previous list
in order to not cut the existing communication channels.
But with the new display of "unsupported wallet" this behaviour
can corrupt the list. An "unsupported wallet" can become
"supported" because user switch app or unlock.
It is preferable to reset the full list and drop the objects
after a refresh.
58a3d21eff766627bde45727f00e101408983602 contrib: update the package pins in the repro build Dockerfiles (Antoine Poinsot)
0bd3bd4d2ef8f88bde2f9abbeed8c1d68408801b Update Github links to wizardsardine/liana (Antoine Poinsot)
ca05bd3fbc2862d3f219491c396b7c1e21b83a99 gui: release 0.2 (Antoine Poinsot)
Pull request description:
Based on #310. Needs to be updated once it is merged.
ACKs for top commit:
darosior:
self-ACK 58a3d21eff766627bde45727f00e101408983602 -- did a full release build on this branch.
Tree-SHA512: 3b7c5a9a5df154dbce5f3615143e8e251c3155a493dc62758ce21a59a640befe469445a171774dcd8e6e1166c9ca0613e9c076a7b073b5b847eb025c13ab56af
3a573b695e07d357e844dd9d573d578e05ecebed qa: test we discard RBF'd deposits (Antoine Poinsot)
f1532f8afcceb5ded299f5ddc425c0473de49871 bitcoin: track expired unconfirmed deposits, remove them from DB (Antoine Poinsot)
ed156543c95db2e6e1fd4712ec7dcb0dfc025c41 database: permit to remove coins from DB (Antoine Poinsot)
62c4b9a01c16c60f66db99abab967efa4e0429dd bitcoind: cleanup gettx cache and conflict detection in spent_coins (Antoine Poinsot)
201c9f21b60dbc5ff0afd9aa73c3a011c08e4459 bitcoin: cache calls to 'gettransaction' when checking for coins confirmation (Antoine Poinsot)
13214c887f373fbe953c01aa8d3a31ae9b65ee98 qa: test we discard RBF'd deposits when replacement is confirmed (Antoine Poinsot)
Pull request description:
Previously we would not detect whether the transaction for an unconfirmed deposit was still in our mempool. For instance a deposit that was RBF'd would result in us storing two coins, the replaced and the replacement. Keeping the unconfirmed replaced coin forever.
Fix this by dropping unconfirmed coins from our database if their transaction isn't in our mempool (anymore).
Fixes#72.
ACKs for top commit:
edouardparis:
utACK 3a573b695e07d357e844dd9d573d578e05ecebed
Tree-SHA512: 677bedbf8024eb16e5f97264c2e01d733732673548772fc3a97868b9ca42a0429ad8c0d888b05d3bb2db9b85d309c418b916c9af578f54e499b5d0689376a1b0
This is the same logic as the (cleaner) one we introduced in
confirmed_coins. Merge the two in a single CachedTxGetter, unfortunaly
at the expense of having to clone() the GetTxRes for now because i don't
want to fight with the borrow checker.
0e05ff5244276f3ae02d2bb264479106ccfc4222 gui: change log_level of ledger_transport_hid (edouard)
Pull request description:
ledger hid apdu exchanges were displayed in the log because of ledger_transport_hid crate.
This commit removes these log entries.
ACKs for top commit:
edouardparis:
Self-ACK 0e05ff5244276f3ae02d2bb264479106ccfc4222
Tree-SHA512: 3961e38dabfe850d10c9670dcb8e89c60bad4ee6267ff4ec93929901d32bd53ddbe0f328c18346297749921fd9e9dcc6ab46ba0aab4cc1476f530bbbb36540cd
5e79c05c544a573a9030e64564c412d95ac34f57 Add patch to liana version (edouard)
Pull request description:
The struct will be imported by the gui which
may need one day to increase the patch version number.
ACKs for top commit:
darosior:
ACK 5e79c05c544a573a9030e64564c412d95ac34f57
Tree-SHA512: 8f129587c638107be31e9a1aa00387ba8d319338df1f3a1072386585901476dd677c85fa3fb8e7a985f11ea6cc2d5607279554ad5a2d651e2c14dda3842353a4
7a3eeb741ea42e3af4d5414e6778bb3fa585f762 ci: test the GUI against it's current MSRV (Antoine Poinsot)
b2602e02046c21636c86b71247a1988ee71616bb contrib: use unpublished rustc 1.64 to build the GUI in Guix (Antoine Poinsot)
0a1c04db3625608522dba98bc4d2a070853cdd1a contrib: fixup patching in Guix build script (Antoine Poinsot)
3f49366b86dc4e2f10fd9e415576f794937a01dd contrib: update the Iced patch for reproducible builds (Antoine Poinsot)
Pull request description:
The Iced dependency was updated, and the patch wasn't valid anymore.
ACKs for top commit:
darosior:
ACK 7a3eeb741ea42e3af4d5414e6778bb3fa585f762 -- tested a reproducible build with this.
Tree-SHA512: 4e79821f24d9be8b5b2db5f415372f6a0f54ebff616d6335f38569ff047786c3e38b3f34e9f85b5141451af13f86651e827d176f01b4f51d12dd4dcc69beb434
ca39b15edd366d23d4bcf7433273663807fa2aab installer: generate multiple xpubs in participate step (edouard)
445ad733fbb6604f2b1acfc93db454eb4794baf7 Add signatures information to spend (edouard)
bf1e9e4b808a7c275766975467e637b883524236 gui: new module settings (edouard)
689f19a4f22009a9043a7308e00c96bf1de97a45 Edit key name in installer (edouard)
bcd223f3fb8cf7224eede75b5cfeb97ae8e93ef9 Add sigs number and threshold to spend list view (edouard)
1f3399bd0f78593a689f0ec50934b1cf4f34f692 Add PartialSpendInfo to SpendTx model (edouard)
9a1cda2f5ed80a66324b272b7cf4c3bd7202e953 Use spend state in recovery panel (edouard)
a2ac34e6b03ef92025ac0f9c9cb709719ea6375a installer: Participate in a new wallet section (edouard)
dfc10eba61818b87ef12f057507a9b9660851d61 Add multisig wallet creation in installer (edouard)
Pull request description:
ACKs for top commit:
edouardparis:
Self-ACK ca39b15edd366d23d4bcf7433273663807fa2aab
Tree-SHA512: 67898dd5cc9ddfe135ddd2e57c6f3d6a6a10cf63211fe44c35e36cd182d4023cdefb8b15d2bd07290d83d631e360b20114b60e85728f9bbdc8c0e0d16b37f746
1cd61c9236b7ecacd39787a84edfcacaec024673 descriptors: require that an origin be set for the keys. (Antoine Poinsot)
Pull request description:
It would be possible for users to create a descriptor with xpubs without an origin set. In fact, not many are used to origins and it's a very likely mistake. Signers need this information in order to be able to sign. So they could potentially create a wallet and potentially never be able to sign for one or multiple keys.
Fix this by requiring an origin for all keys in the descriptor.
Fixes#290.
ACKs for top commit:
edouardparis:
ACK 1cd61c9236b7ecacd39787a84edfcacaec024673
Tree-SHA512: fb17f2fb11a467289cb942b2df2a6331d03d52805ab02bf39eeb1f3086d4110d9e3cea3f12e97b0822af8879b483eb51b02eb17b70595a0b199e5dcbbb7229dd
