887b71367bebe277740443ff4e2d0f1470d2c0ab Order history txs in home state (edouardparis)
Pull request description:
ACKs for top commit:
edouardparis:
Self-ACK 887b71367bebe277740443ff4e2d0f1470d2c0ab
Tree-SHA512: f091f29b1c0b96b80fbacc40ece131edce2d2bfd571cc239b84295c59880e796cb787c3c086b37adb67bef2a0ae428dd86d4a274b1b51208fe90bde77aceb858
0f48db7a88b1dcaee68190a6a6ee5e73b5630f01 electrum: check connectivity before creating client (Michael Mallan)
Pull request description:
This is to resolve https://github.com/wizardsardine/liana/issues/1276.
Tested that this change stops GUI freezing in https://github.com/wizardsardine/liana/pull/1241 when using invalid address in Electrum settings.
ACKs for top commit:
darosior:
utACK 0f48db7a88b1dcaee68190a6a6ee5e73b5630f01
Tree-SHA512: b176ceafda887c4ef97110c6a87807787136da6fd67fb8a2af0b63c102a7d0145f46df5d75b61a10b45cf4cbc91ad27535de7abaf13409e259d1c75881abcef3
819eb920c0210b9d95689f40cfbeba65c03280af gui(settings): allow to change node type (Michael Mallan)
2381227216663a6a2336ee79905646fb2a3aadca gui(settings): view & edit Electrum settings (Michael Mallan)
b570039ff8e740be43c6aebdc62910f4ab4eac05 gui(settings): rename Bitcoin Core to Node (Michael Mallan)
db20ae4b677cf4ffdf19ad0df6f50ea983b34a02 gui(installer): reduce empty space height (Michael Mallan)
0993905879629c58da24a1145acbb49faf31da3a gui(installer): update wording to include Electrum (Michael Mallan)
f40af570bccecf9361410df334ad30e45f57659b gui(installer): split long string and run cargo fmt (Michael Mallan)
0f09be151ca3ba1353d23b170805c845fc8ab5cc gui(installer): don't change values while waiting for ping result (Michael Mallan)
c93aa88d74620a555bd442bc3504e12af198f00a gui(installer): add electrum node option (Michael Mallan)
341e4467dbf727d78f31690b8dbc405995ab4088 gui(installer): allow for different node types (Michael Mallan)
83172c7bc584283d7dd4b93f3f18ec0f2d9a1ad0 gui(installer): add general node definition (Michael Mallan)
046b54e6a9337aaf478ea6e61f2d28d73f1b15c1 gui(installer): define bitcoind from general node struct (Michael Mallan)
c5d9d007fb908308592635b8f85236a1b7a4b5ae gui: move bitcoind to new node module (Michael Mallan)
4536eff561459648cbf0666ec757db95ac29de4f gui(installer): extract logic for try ping bitcoind (Michael Mallan)
ef44cf329adc0eed01451950454edfb7e54be515 gui(installer): add module for node step (Michael Mallan)
f74f071b8a3e4eda77cc5b468f891b21c5e4e2a4 gui: upgrade liana dependency (jp1ac4)
Pull request description:
This is for #1223.
For now, it's possible to edit the node's settings but not to change node type.
Remaining tasks:
- [x] Revert Cargo.toml once #1222 is merged.
- [x] Update wording as per https://github.com/wizardsardine/liana/issues/1223#issuecomment-2286483134.
ACKs for top commit:
pythcoiner:
ACK 819eb920c0210b9d95689f40cfbeba65c03280af
Tree-SHA512: 362a14d32c2e13ba286d252d9f8a1106d63e5c40198776653b0623b433435329663126307e17da017fdbbd8a8ad273b703cc3ba54fd13fa5a0afd7dd9179089a
cc02ac043b647b22b0533dec3f99cbcf62e7ee8f installer: Go directly to next step once login (edouardparis)
Pull request description:
Once the user is connected we redirect him to
the next step in the install process, but
we keep the Connected state in case the user
want to edit email after clicking on Previous
As discussed in #1240
ACKs for top commit:
edouardparis:
Self-ACK cc02ac043b647b22b0533dec3f99cbcf62e7ee8f
Tree-SHA512: c2b65288fd8fcb07edf021ad45391fc1761d185b7da48985a2108be5dce5a1f744fa37da8c667cb140df21ce2c4403492fab58c11098a3695c5c4b3496219346
Once the user is connected we redirect him to
the next step in the install process, but
we keep the Connected state in case the user
want to edit email after clicking on Previous
As discussed in #1240
7cce13656266d90431133ccb4de5ccf086f4aa82 qa: test proc stderr isn't None before closing it (Antoine Poinsot)
ba553e9ddb104af454d9eff9969c0e37a1f3061a qa: adapt bitcoind config to support 28.0 (Antoine Poinsot)
Pull request description:
ACKs for top commit:
darosior:
ACK 7cce13656266d90431133ccb4de5ccf086f4aa82
Tree-SHA512: ff0a9addd6063cd31b8e7f76cbf2b7ed1f5e85c2d7192ed5e73bba1c81a3bb4e8740ff7c5242d054115e28e484c87f85cafcbbc4dcc4ecf24ff415a028e1b0b5
341f9406645dd26c7862ae7abe28aed5c7e9185d func test: prevent disconnects when using mocktime (Michael Mallan)
b630d46770319dce051c09133a528b1ff322a358 func test: wait for block heights to match (Michael Mallan)
ada89c5c56090338554cb8db0e078674798341ac doc: update with electrum info (Michael Mallan)
177dfc46c305ea7ab1c421e9a9d60535327fedff lib: expose BDK's electrum client (Michael Mallan)
72c63148984bc69ff7c4b04cfdc16a12994b2956 ci: add electrs to func tests (jp1ac4)
371e31e3f35ac5054fe2bd7dd04bcdd8790e8988 func test: allow to run using electrs backend (jp1ac4)
a85d4887e9efeaada6a1a79f4a35a32cf8444569 func test: allow for different bitcoin backends (jp1ac4)
1b04b294910d1cb65885f17dd266029771b476cf func test: fix min rbf feerate (jp1ac4)
c7ee8620204e72bcf990a994c1bdf89652f40c84 bitcoin: add electrum backend (jp1ac4)
c4c2424f244fcd4461cf5378f83e9abe4ba12866 bitcoin: expose MempoolEntryFees (jp1ac4)
5011ad929037af294dc471844d8bf5c799040383 bitcoin: return spent block height & time separately (jp1ac4)
689442c2c910ad5967651c0cc9f84bdc71cd0002 bitcoin: allow to store UTXO deriv index (jp1ac4)
4c02b0d43115f5d792908dd30580eaf241e8fc58 bitcoin: use mut ref for start_rescan (Michael Mallan)
69259c12362b2be8da8f210def1d3e338fa8ab00 poller: sync wallet before checking updates (jp1ac4)
89e004d4fdd88324da255bc85aeb8adbb90653b1 bitcoin: add sync_wallet method to interface (jp1ac4)
34b9a49328e7a3313ad6f4a0537ac5b349b54d76 config: add general bitcoin backend option (jp1ac4)
e267f66be6ce3bebac413ee9e4901d27e8cbe9df descriptors: allow to get underlying public key (jp1ac4)
Pull request description:
This is to add the Electrum backend as part of https://github.com/wizardsardine/liana/issues/56#issuecomment-2183063784.
This requires the database to be running version 5 following #1180. The migration from a previous DB version must be done using bitcoind. Thereafter, the daemon can be run using an Electrum backend by replacing the `[bitcoind_config]` section in the daemon.toml config file with:
```
[electrum_config]
addr = "127.0.0.1:50001" # adjust IP:port as required
```
Remaining tasks:
- [x] Include ancestors and descendants when getting a transaction's `MempoolEntry`
- [ ] Additional sanity checks, e.g. check Electrum version
- [x] Check if logic regarding ongoing rescan needs to be adjusted
- [x] Add Electrum backend to functional tests
- [x] Add Electrum backend to CI pipline functional tests
- [ ] Add unit tests
- [x] Update README & other docs
ACKs for top commit:
darosior:
re-ACK 341f9406645dd26c7862ae7abe28aed5c7e9185d
Tree-SHA512: dbbb375123cc5c566f5c8535d002ac4491b4be5ab2766845b5b7ab51d19e7e85eafc0097235e1ebc5c5a049bd2222ab5582264314e46c77b5fff8027da31b803
Here, the min RBF feerate is 1 more than that of the transaction
to be replaced.
The feerate of the transaction to be replaced may vary slightly
depending on the signature size and is calculated during the test.
As such, the min feerate should be set according to this
calculated value.
7c571370bd599e294b161812e6b8940c64012744 gui: filter out logs for `cosmic_text` (pythcoiner)
Pull request description:
Logs are litterally flooded by `cosmic_text` logs at DEBUG level, this PR filter out this target
ACKs for top commit:
edouardparis:
ACK 7c571370bd599e294b161812e6b8940c64012744
Tree-SHA512: 252490eafabe6e56dbe5e20d0f0721a9d068756d0443e47693c6448e50c2f11121b8c9d8c114bb4fa38bd0b746a118f7d3133e4236e0759d5b27b63ac9c3907e
1d720d12a48be082668570a0448a9e3fafcf230c Use different msg for reloading or extending history tx list (edouardparis)
Pull request description:
After reorg last payments were not removed as the reloading message was handle as an extension of the list.
close#1260
ACKs for top commit:
pythcoiner:
ACK 1d720d12a48be082668570a0448a9e3fafcf230c tested w/ bitcoind backend
Tree-SHA512: 8cdb68cec969bbc187e254e05f29a847e26cf6ba198f11c8d11ddccc64f19b3b38a8d4f3dff30eb60cf7779df378ed241986bc50daba8bfb11b9c22b2139b846
877b95e52ddb3871f0469253d121a157f028798d Fix RemoteBackend::Undefined panic because of skipped step (edouardparis)
Pull request description:
Because Testnet and regtest do not ask user the choice of backend the install panic as the RemoteBackend enum stays to undefined
close#1264
ACKs for top commit:
edouardparis:
Self-ACK 877b95e52ddb3871f0469253d121a157f028798d
Tree-SHA512: 80b3ca99e8848bb2025174874307e6931947548c5964c7a6dd46517becb40c7a9fdd5a51385d24167541374ce8219bb6d04d96a20a8b3d8aefcef6a936346025
d497c4dfe330b759cffe7f2e06b1a1fe90b58da1 Change name displayed in launcher (edouardparis)
2c79e618ce33120d63459d8be179c34807411bfc Use Liana Connect terminology (edouardparis)
Pull request description:
discussed in #1240 and discord
ACKs for top commit:
edouardparis:
Self-ACK d497c4dfe330b759cffe7f2e06b1a1fe90b58da1
Tree-SHA512: 49e61b47e1eb13dbde3195819f845ae715d3ee2b6f84136c5ace53a77fef8b6fda4a05c940d6c6d7e0148690c1ce87bbc8e166d31aa6d710216a8379a938000a
