forked from dashpay/dash
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge bitcoin-core/gui#121: Early subscribe core signals in transacti…
…on table model cafef08 qt: Refactor to remove unnecessary block in DispatchNotifications (João Barbosa) 57785fb qt: Early subscribe core signals in transaction table model (João Barbosa) c6cbdf1 qt: Refactor ShowProgress to DispatchNotifications (João Barbosa) 3bccd50 qt: Set flag after inital load on transaction table model (João Barbosa) Pull request description: This fixes the case where transaction notifications arrive between `getWalletTxs` and `subscribeToCoreSignals`. Basically notifications are queued until `getWalletTxs` and wallet rescan complete. This is also a requirement to call `getWalletTxs` in a background thread. Motivated by bitcoin#20241. ACKs for top commit: jonatack: tACK cafef08 ryanofsky: Code review ACK cafef08. Only change since last review is splitting commits and replacing m_progress with m_loading. meshcollider: Code review ACK cafef08 Tree-SHA512: 003caab2f2ae3522619711c8d02d521d2b8f7f280a467f6c3d08abf37ca81cc66b4b9fa10acfdf34e5fe250da7b696cfeec435f72b53c1ea97ccda96d8b4be33
Showing
1 changed file
with
32 additions
and
30 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters