Skip to content
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.

Sign up for GitHub

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

Jump to bottom

nodebridge - enumerate while other call waiting for button request #14771

Closed
mroz22 opened this issue Oct 8, 2024 · 6 comments · Fixed by #15142
Closed

nodebridge - enumerate while other call waiting for button request #14771

mroz22 opened this issue Oct 8, 2024 · 6 comments · Fixed by #15142
Labels
Bridge -> nodeUSB

Comments

@mroz22
Copy link
Contributor

mroz22 commented Oct 8, 2024 •
edited
Loading

client 1:

  1. get address
  2. show on device, do not confirm and wait

client 2:

  1. open app
  2. enumerate does not respond - gets aborted, transport-error is emitted resulting in some troubleshooting screen
@github-project-automation github-project-automation bot moved this to 🎯 To do in Suite Desktop Oct 8, 2024
@mroz22 mroz22 mentioned this issue Oct 31, 2024
Merged
@evgenysl
Copy link

@mroz22 results for both Bridge 2.0.33 and NodeBridge 3.0.0

fix branch

When I click "Use Trezor here", I'm correctly switched to web Suite:

fix branch 2

@evgenysl

This comment was marked as resolved.

Sign in to view
@evgenysl
Copy link

evgenysl commented Nov 4, 2024

T1 and TS5 are connected.

client 1:

  1. get address
  2. show on TS5, do not confirm and wait

client 2:

  1. open app
  2. Only T1 is visible, TS5 is not discovered

m2

@evgenysl
Copy link

evgenysl commented Nov 4, 2024 •
edited
Loading

NodeBridge
Desktop client 1:

  1. get address
  2. show on device, do not confirm and wait

Web client 2:

  1. open app
  2. click Use Trezor here
  3. Wallet is correctly switched
  4. Bitcoin -> Receive -> Show full address ✅

m4

m5

@evgenysl
Copy link

evgenysl commented Nov 4, 2024

T1 and TT are connected.

m20

  1. Disconnect TT cable
  2. Correct TT disappeared from wallet switcher ✅
  3. Connected TT back without closing wallet switcher
  4. TT appeared in wallet switcher

Tried this many times, disconnecting different wallets (active and inactive in wallet switcher).

@evgenysl
Copy link

evgenysl commented Nov 4, 2024

T1 and TT are connected.

  1. T1 wallet is active
  2. Go to bitcoin account
  3. Disconnect T1 cable
  4. Suite switches to TT correctly

@github-project-automation github-project-automation bot moved this from 🎯 To do to 🤝 Needs QA in Suite Desktop Nov 5, 2024
@evgenysl evgenysl moved this from 🤝 Needs QA to ✅ Approved in Suite Desktop Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bridge -> nodeUSB
Projects
Suite Desktop
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(transport-bridge): synchronize was too strict, allow enumerate an… trezor/trezor-suite
2 participants
@mroz22 @evgenysl