Skip to content

CP-12074: Add ledger account#3519

Closed
Copilot wants to merge 1 commit intoray/cp-12074from
copilot/sub-pr-3518
Closed

CP-12074: Add ledger account#3519
Copilot wants to merge 1 commit intoray/cp-12074from
copilot/sub-pr-3518

Conversation

Copy link

Copilot AI commented Jan 28, 2026

Description

Ticket: CP-12074

Implements ledger account creation with multi-chain address derivation and mainnet/testnet support.

Changes:

  • Ledger account creation: Derives EVM/C-Chain, Bitcoin, XP, and optionally Solana addresses from ledger device
  • Mainnet/testnet handling: Pre-derives both network addresses simultaneously to avoid re-prompting users when toggling developer mode
    • Stores address pairs in new ledgerAddresses Redux state
    • AccountsService retrieves correct addresses based on current network mode
  • UI improvements:
    • LedgerConnectionCaption component displays device connection instructions
    • "Add account" button conditionally shown when Avalanche app is detected on device
    • Solana "Enable" button appears when Solana app is open, triggers derivation flow
  • State management:
    • New LedgerAddressesCollection type tracks mainnet/testnet addresses per account
    • setLedgerAddresses action populates addresses during account creation
    • addAccount thunk creates both network variants for ledger wallets

Screenshots/Videos

Include relevant screenshots or screen recordings of iOS and Android.

Testing

Dev Testing (if applicable)

  • verify you can see the caption in wallet screen for ledger wallet to instruct user to turn on device and open avalanche app
  • verify you can add account with evm/c-chain, bitcoin, and xp addresses populated
  • verify you can enable and derive solana address in account address screen
  • verify you can go through the solana connection flow when you press enable to derive solana address
  • verify you can see the caption to instruct user to turn on device and open solana app in account address screen
  • verify you see the correct mainnet/testnet addresses when you toggle developer mode

QA Testing (if applicable)

  • Provide instructions for QA to test this feature thoroughly
  • State expected behavior / acceptance criteria

Checklist

Please check all that apply (if applicable)

  • I have performed a self-review of my code
  • I have verified the code works
  • I have included screenshots / videos of android and ios
  • I have added testing steps
  • I have added/updated necessary unit tests
  • I have updated the documentation

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Author

Copilot AI commented Jan 28, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • chromedriver.storage.googleapis.com
    • Triggering command: /usr/local/bin/node /usr/local/bin/node install-npm.js (dns block)
  • https://api.github.com/repos/yarnpkg/yarn/releases
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /tmp/xfs-a4a103df/.yarn/releases/yarn-1.22.19.cjs set version classic --only-if-needed (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Add ledger account functionality CP-12074: Add ledger account Jan 28, 2026
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