Skip to content

Boyer/ledger app connect flows#3361

Merged
ruijialin-avalabs merged 75 commits intomainfrom
boyer/ledger-app-connect-flows
Jan 23, 2026
Merged

Boyer/ledger app connect flows#3361
ruijialin-avalabs merged 75 commits intomainfrom
boyer/ledger-app-connect-flows

Conversation

@B0Y3R-AVA
Copy link
Copy Markdown
Contributor

@B0Y3R-AVA B0Y3R-AVA commented Nov 21, 2025

Description

iOS: 7146
Android: 7147

Ticket: CP-11829

Please provide:

  • A summary of the changes
  • Motivation and context for this change
  • Any dependencies introduced
  • (If breaking) migration steps and instructions

Screenshots/Videos

Include relevant screenshots or screen recordings of iOS and Android.

android-app-connection-flows-compressed.mov

Testing

Dev Testing (if applicable)

  • Go to import wallet screen
  • Select Import ledger
  • when going through flow confirm that the connection flows match designs

  • turn off ff for solana
  • make sure you don't get the prompt to connect solana

  • keep the solana ff on
  • skip solana connect in the conenection flow
  • make sure the wallet is created successfully
  • check the app to make sure solana does not appear anywnere in the app (receive screen, portfolio, swap, buy)

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

@B0Y3R-AVA B0Y3R-AVA force-pushed the boyer/ledger-app-connect-flows branch from c7df4e4 to e94553b Compare November 21, 2025 01:43
@B0Y3R-AVA B0Y3R-AVA marked this pull request as ready for review November 21, 2025 18:25
@B0Y3R-AVA B0Y3R-AVA force-pushed the boyer/ledger-app-connect-flows branch from d6789f6 to 1bfc203 Compare January 7, 2026 15:34
Comment thread packages/core-mobile/app/new/features/ledger/contexts/LedgerSetupContext.tsx Outdated
Comment thread packages/core-mobile/app/store/account/listeners.ts Outdated
Comment thread packages/core-mobile/app/services/ledger/LedgerService.ts Outdated
Comment thread packages/core-mobile/app/services/account/AccountsService.tsx Outdated
@B0Y3R-AVA B0Y3R-AVA force-pushed the boyer/ledger-app-connect-flows branch from cbdd9ad to ebff503 Compare January 7, 2026 21:30
Comment thread packages/core-mobile/app/new/features/ledger/screens/CompleteScreen.tsx Outdated
Comment thread packages/core-mobile/app/new/features/ledger/hooks/useLedgerWallet.ts Outdated
Comment thread packages/core-mobile/app/new/common/components/ProgressDots.tsx
Comment thread packages/core-mobile/app/new/common/components/ScrollScreen.tsx Outdated
@B0Y3R-AVA B0Y3R-AVA force-pushed the boyer/ledger-app-connect-flows branch 2 times, most recently from 66b0aca to d95df00 Compare January 15, 2026 19:36
Comment thread packages/core-mobile/app/services/wallet/LedgerWallet.ts Outdated
Comment thread packages/core-mobile/app/services/account/AccountsService.tsx Outdated
Comment thread packages/core-mobile/app/services/ledger/LedgerService.ts Outdated
Comment thread packages/core-mobile/app/services/ledger/LedgerService.ts
Comment thread packages/core-mobile/app/new/features/ledger/consts.ts
@ruijialin-avalabs ruijialin-avalabs self-requested a review January 21, 2026 14:25
@ruijialin-avalabs ruijialin-avalabs force-pushed the boyer/ledger-app-connect-flows branch 2 times, most recently from 47d363d to c5ec421 Compare January 23, 2026 13:48
Copy link
Copy Markdown
Contributor

@alexnicolae-ava alexnicolae-ava left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dev tested, great work! 🔥

@ruijialin-avalabs ruijialin-avalabs merged commit 5b9af49 into main Jan 23, 2026
4 checks passed
@ruijialin-avalabs ruijialin-avalabs deleted the boyer/ledger-app-connect-flows branch January 23, 2026 15:15
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