trezor / trezor-suite

Trezor Suite Monorepo
https://trezor.io/trezor-suite
Other
695 stars 246 forks source link

Device connected status after abort of dry run #12756

Open bosomt opened 3 months ago

bosomt commented 3 months ago

Describe the bug You can get Device connected status in app after abort of dry run abd only way to get rid of it is to reconnect device. See video

Steps to reproduce:

  1. Start dry run or multishare backup
  2. Abort during entering of words
  3. Observe suite behavior

Info:

Screenshots: https://youtu.be/ZJzNUs83hzw image

peter-sanderson commented 2 months ago

This happens because of this in FW: https://satoshilabs.slack.com/archives/CL1D61PQF/p1718029051129309

komret commented 1 month ago

Needs to be fixed in firmware, closing here. @bosomt is there an issue in the firmware repo?

matejcik commented 1 month ago

to be clear

while in recovery state, we can make FW reply with error to anything but RecoveryDevice -- as discussed on Slack above

but I fail to see how it resolves the problem in @bosomt's video....? what is the firmware-side problem you want solved here?

komret commented 1 month ago

I thought this would be resolved by the changes to FW proposed in Slack. You are right the SUite screen is unrelated to firmware.

komret commented 1 month ago

Firmware issue can be closed, sorry.