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

Add KLS to database #265

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Conversation

lemois-1337
Copy link
Contributor

@tdejoigny-ledger
Copy link
Contributor

Hi @lemois-1337 ty for you PR
But it seems that Flex device is missing

@lemois-1337
Copy link
Contributor Author

@tdejoigny-ledger Added next-gen Ledger Flex device, was forgotten in first commit, force pushed.

@tdejoigny-ledger
Copy link
Contributor

tdejoigny-ledger commented Oct 7, 2024

sorry @lemois-1337 my bad, I missed that you have also nanoS. NanoS shall be removed.
New app should not be deployed on NanoS (EOL product). You can also remove NanoS in the ledger_app.toml and add Flex

lemois-1337 added a commit to karlsen-network/app-karlsen that referenced this pull request Oct 8, 2024
Updated supported devices list as mentioned in comment:
LedgerHQ/ledger-app-database#265 (comment)

Nano S is EOL while Flex is the next-gen Ledger device.
@lemois-1337
Copy link
Contributor Author

@tdejoigny-ledger I've removed NanoS from this PR (force-pushed). I've fixed it in ledger_app.toml as well and thanks for letting me know.

@tdejoigny-ledger tdejoigny-ledger merged commit 67f95a7 into LedgerHQ:main Oct 8, 2024
2 checks passed
@tdejoigny-ledger
Copy link
Contributor

perfect ty @lemois-1337

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants