Differential D16360

[electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode
ClosedPublic
Actions

Authored by PiRK on Jun 20 2024, 16:23.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC2e3c49b5123a: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver…

Summary

When the BCH app is open for a while and the device goes to screensaver mode, and Electrum ABC is restarted with the Ledger wallet, it would print an unhelpful 'invalid status 5515' error. To recover, the user would have to unlock the device and close/reopen the wallet.
Handle this status with a more useful error dialog.

Test Plan

Test the above scenario

Diff Detail

Repository

rABC Bitcoin ABC

Branch

status_errorswq

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 29363
Build 58262: Build Diff	electrum-tests
Build 58261: arc lint + arc unit

Event Timeline

PiRK created this revision.Jun 20 2024, 16:23

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 20 2024, 16:23

PiRK requested review of this revision.Jun 20 2024, 16:23

Harbormaster completed remote builds in B29363: Diff 48335.Jun 20 2024, 16:25

PiRK retitled this revision from [electrum] handle 0x5515 status when device is locked in screensaver mode to [electrum] handle 0x5515 status when a Ledger device is locked in screensaver mode.Jun 20 2024, 16:25

Fabien accepted this revision.Jun 20 2024, 16:52

This revision is now accepted and ready to land.Jun 20 2024, 16:52

Closed by commit rABC2e3c49b5123a: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver… (authored by PiRK). · Explain WhyJun 21 2024, 08:13

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC2e3c49b5123a: [electrum] handle 0x5515 status when a Ledger device is locked in screensaver….