Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

[use-contractkit] Ledger Error Message #66

Open
patricknbaron opened this issue Nov 16, 2021 · 5 comments
Open

[use-contractkit] Ledger Error Message #66

patricknbaron opened this issue Nov 16, 2021 · 5 comments
Assignees
Labels
Applications bug Something isn't working focus:reliability Internal Refactors for reliability v4.1 Scheduled to be included in the release right after v4

Comments

@patricknbaron
Copy link

When a user connects with a Ledger but opens another network like Ethereum on the app, the error message is incorrect.

Current message: "Possible connection is lost with the ledger. Check if still on and connected. Ledger device: UNKNOWN_ERROR (0x6804)"

Should be something like: "You're connected to Ethereum network, switch to the Celo app on your Ledger."

image

@dckesler
Copy link
Contributor

dckesler commented Dec 8, 2021

It seems that when Ledger updated their software the Celo app became sort of buggy. We're looking more into the Celo Ledger app.

@aaronmgdr
Copy link
Member

I think this might be deeper in the stack

@jmrossy
Copy link
Contributor

jmrossy commented Mar 20, 2022

It's definitely deeper in the stack as @aaronmgdr said. The errors Ledger's libs surface are usually pretty terse. We may be able to distinguish between them in some cases but not always

@nicolasbrugneaux nicolasbrugneaux added the bug Something isn't working label May 11, 2022
@nicolasbrugneaux nicolasbrugneaux self-assigned this May 11, 2022
@nicolasbrugneaux nicolasbrugneaux changed the title Error Message [use-contractkit] Ledger Error Message May 11, 2022
@dckesler dckesler moved this to Todo in react-celo 4.1 May 11, 2022
@dckesler dckesler added the v4 Scheduled to be included in v4 release label May 11, 2022
@aaronmgdr
Copy link
Member

ledger package we use is a bit old we should see if upgrading helps https://github.com/LedgerHQ/ledgerjs

@aaronmgdr aaronmgdr moved this from Todo to In Progress in react-celo 4.1 May 19, 2022
@aaronmgdr
Copy link
Member

Depends on 9584

@aaronmgdr aaronmgdr removed the v4 Scheduled to be included in v4 release label May 19, 2022
@aaronmgdr aaronmgdr added focus:reliability Internal Refactors for reliability v4.1 Scheduled to be included in the release right after v4 labels Jun 1, 2022
@aaronmgdr aaronmgdr moved this to In Progress in react-celo 4.1 Jun 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Applications bug Something isn't working focus:reliability Internal Refactors for reliability v4.1 Scheduled to be included in the release right after v4
Projects
Status: In Progress
Development

No branches or pull requests

5 participants