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

[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 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
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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