Skip to content

[Bug]: Rejecting location permission when pairing Ledger Nano X #8643

@plasmacorral

Description

@plasmacorral

Describe the bug

In the add hardware flow for Ledger on Android, if the user rejects the location permission request, it is simply requested a second time. When the user rejects a second time, then MMM looks for a Nano with a spinning wheel until MMM closes. On relaunch of MMM, the user is returned to Connect Ledger with an indefinite spinner.

Expected behavior

If user rejects a necessary permission, then an error message is gracefully presented with suggested actions for the user.

This is failing at Scenario 3.H in that no error message is being presented.

Screenshots/Recordings

Sped up recording: https://recordit.co/XKMLZz0BDt
Half speed recording to see the second permission request: https://recordit.co/vqiQb7KQgS

Steps to reproduce

  1. From Android settings>connected devices> see all make sure LNX not currently paired
  2. Have LNX unlocked and the Ethereum app opened
  3. Unlock MMM
  4. Tap accounts menu from wallet view
  5. Select add account or hardware wallet
  6. Tap Add hardware wallet
  7. Tap Ledger
  8. When location permission is requested, decline it
  9. When location permission is requested again immediately, decline it again
  10. Note that app will close
  11. Relaunch and observe peristant spinner next to Looking for device

Error messages or log output

No response

Version

7.17.0 RC1 build 1266

Build type

None

Device

Pixel 5a android 14

Operating system

Android

Additional context

Location permission not required on iOS

Tested with Ledger Nano X hardware v2.0, firmware 2.2.3, micro controller 2.30, bootloader 1.16
LNX Ethereum version 1.10.3

Severity

No response

Metadata

Metadata

Assignees

Labels

LedgerLedger hardware wallet related issue or developmentPriority - HighTask with high priorityQA PassedQA testing has been completed and passedSev2-normalAn issue that may lead to users misunderstanding some limited risks they are takingregression-RCDEPRECATED: Please use "regresssion-RC-x.y.z" label insteadregression-RC-7.17.0release-7.17.0Issue or pull request that will be included in release 7.17.0release-7.21.0Issue or pull request that will be included in release 7.21.0team-hardware-wallets-deprecatedDEPRECATED: please use "team-accounts-framework" insteadtype-bugSomething isn't working

Type

No type

Projects

Relationships

None yet

Development

No branches or pull requests

Issue actions