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

Updated ledger logo #1691

Open
wants to merge 16 commits into
base: canary
Choose a base branch
from

Conversation

MikelCalvo
Copy link

This PR aims to fix the following issue: #1647

The assets have been downloaded directly from https://developers.ledger.com/docs/connectivity/ledgerJS/faq/ledger-logos

Let me know If I missed any other icon.

mholtzman and others added 16 commits August 31, 2023 13:11
* start migrating types to use zod for parsing and defaults

* finish basic state loading, add some tests

* use current base state when state is corrupted

* fix tests

* fix gas for tests

* update legacy migrations to be more resilient

* add test migration files

* update migration 38

* update tests

* continue updating schemas

* add dapp schema

* add more types

* initial working version

* fix some tests

* finish tokens schema

* finish updating migrations

* add settings for panel and selected

* update most of state not to persist

* allow chain metadata with no native currency rate data

* update types to support migration from v0.4.4

* fix typo

* add last migration test

* fix test

* dont persist signers

* update test

* fix test

* final changes

* update version

* remove no check and add any to legacy migrations

* skip test

* cleanup
* block out workspace ui and interactions, add examples
* fix linux SIGTRAP related to power monitor

* update tests

---------

Co-authored-by: Jordan Muir <[email protected]>
@mholtzman mholtzman force-pushed the canary branch 2 times, most recently from e042db7 to 066b9a9 Compare March 12, 2024 16:28
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.

3 participants