-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Update LibWallet recovery task event handling #3142
fix: Update LibWallet recovery task event handling #3142
Conversation
PR queued successfully. Your position in queue is: 1 |
PR failed to merge with reason: Some CI status(es) failed |
PR queued successfully. Your position in queue is: 1 |
PR failed to merge with reason: Some CI status(es) failed |
The recovery task broke out of its monitoring loop before getting the `UtxoScannerEvent::Completed` event. This PR just moves that break statement so that the final completed callback is made. Also Ignore `test_store_and_forward_send_tx` due to it being flakey on CI and the functionality is covered by Cucumber tests.
5a55aa8
to
263e87f
Compare
PR queued successfully. Your position in queue is: 1 |
PR failed to merge with reason: Some CI status(es) failed |
PR queued successfully. Your position in queue is: 1 |
Description
The recovery task broke out of its monitoring loop before getting the
UtxoScannerEvent::Completed
event. This PR just moves that break statement so that the final completed callback is made.How Has This Been Tested?
Wallet clients will need to test this
Checklist:
development
branch.