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.

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

Onboarding - Select-wallet-type modal not displayed after completing onboarding (and turning passphrase on) #3592

Closed
sorooris opened this issue Mar 31, 2021 · 4 comments · Fixed by #3687
Assignees
Labels
bug Something isn't working as expected onboarding Onboarding part for a new user/device

Comments

@sorooris
Copy link
Contributor

Describe the bug
The Select wallet type modal is not displayed after completing onboarding in which passphrase is turned automatically on; compared to the flow when an already set up device is connected after which the modal is displayed and Standard or Hidden wallet selection is offered. I would expect that the modal is displayed every time if passphrase is ON.

Info:

  • Remembered Wallet: no
  • Suite Version: web 21.3.2 6dbeaf3
  • Browser: Chromium 89.0.4389.82
  • OS: NixOS 20.09
  • Bridge: standalone 2.0.28
  • Firmware Version: any (tested model One 1.9.4 ffa96205

Steps to reproduce:
Prerequisite: wiped/reset device.

  1. Launch Suite,
  2. Complete onboarding flow,
  3. Observe.

Expected result:
Wallet type selection is displayed before being redirected to the Dashboard.

Actual result:
Redirected to a Dashboard of a Standard wallet.

Screenshots:
After onboarding flow:
Peek 2021-03-30 16-21
Device setup:
Peek 2021-03-31 13-41

@sorooris sorooris added the bug Something isn't working as expected label Mar 31, 2021
@alex-jerechinsky
Copy link
Member

@matejzak it seems like common sense to have the Select wallet type modal displayed after user finishes onboarding, but I'd appreciate your input first. Is this something that's accounted for in the new onboarding? If so, should we still implement it here?

@matejzak
Copy link

matejzak commented Apr 6, 2021

Heads up to @slowbackspace who is working on the new #3332

@matejzak matejzak assigned slowbackspace and unassigned matejzak Apr 6, 2021
@slowbackspace slowbackspace added the onboarding Onboarding part for a new user/device label Apr 6, 2021
@bosomt
Copy link
Contributor

bosomt commented May 7, 2021

QA OK

Info:

  • Suite version: web 21.5.1 (974429b)
  • Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:88.0) Gecko/20100101 Firefox/88.0
  • OS: MacIntel
  • Screen: 1584x990
  • Device: model T 2.3.6 regular

@sorooris
Copy link
Contributor Author

QA OK

Suite: app 21.5.1 (4d09d88)
OS: macoS Big Sur 11.3
Device: T1 1.9.4 public

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as expected onboarding Onboarding part for a new user/device
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants