-
Notifications
You must be signed in to change notification settings - Fork 315
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
support: add LL token icon #6776
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
3 Ignored Deployments
|
@shaynos is attempting to deploy a commit to the LedgerHQ Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
running pnpm install
on this branch seems to reveal a problem on the importing of that icon
src/native/LL.tsx:13:85 - error TS2322: Type '{ className: string; d: string; }' is not assignable to type 'IntrinsicAttributes & IntrinsicClassAttributes<Pathβ¦
β Property 'className' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<Path> & Readonly<PathProps>'.
β 13 return <Svg viewBox="0 0 1200 1200" height={size} width={size} fill={color}><Path className="prefix__st0" d="M515.156 608.41c-15.329 15.328-15.329 40.774 β¦
β ~~~~~~~~~
β Found 1 error in src/native/LL.tsx:13
β βELIFECYCLEβ Command failed with exit code 2.
ββ Failed in 21.1s at /Users/grenaudeau/dev/ledger-live/libs/ui/packages/crypto-icons
Thanks @gre I've cleaned up the svg, looks like it builds ok now |
Hi @gre friendly reminder on this pr pls |
Hey @shaynos , sorry for the delay. thanks |
Hey @gre no worries - Yes I requested the addition of LightLink token about 2 weeks ago via type form on the provided link. Is there any other actions we need to take? Syncing this now with ledger branch... |
Hi @gre looks like LightLink has now been added to supported tokens here - https://support.ledger.com/hc/en-us/articles/10479755500573-Supported-Coins-and-Tokens-in-Ledger-Live?docs=true Is there anything else I can do to get this one through? |
* develop: (203 commits) ci: update workflow ci: fix ci: updtate e2e workflow π : [BUGFIX] Layout issue with new Gainers/Losers trend widget if LLD is in French or German (#6932) Add basic tests Reorg files into new architecture folder ci: adding changeset ci: ENABLE_TRANSACTION_BROADCAST ci: e2e nightly - setup default value to variables feat(context-module): new module feat(eth): signature for setExternalPlugin is now optional [LIVE-9296] Feature - Use Ethereum app for all EVMs (#6861) support: add LL token icon (#6776) [LLD] - [LIVE-12264] - Add Wallet Sync activation screen (#6782) Add changeset Store components in several folder Initialize WalletSync Manage screen test(wallet-api): add test to avoid future regressions fix(platform): xrp family not present fix(live-12705): display the disabled swap button when the form is not valid ...
β Checklist
npx changeset
was attached.π Description
Add icon for LightLink ERC20 to Ledger
Network: Ethereum Mainnet
Contract Address: 0x0921799CB1d702148131024d18fCdE022129Dc73
Etherscan: https://etherscan.io/address/0x0921799CB1d702148131024d18fCdE022129Dc73
β Context
π§ Checklist for the PR Reviewers