Skip to content

[TASK-10656] feat: remove external wallets#827

Merged
jjramirezn merged 22 commits intopeanut-wallet-devfrom
feat/remove-external-wallets
May 5, 2025
Merged

[TASK-10656] feat: remove external wallets#827
jjramirezn merged 22 commits intopeanut-wallet-devfrom
feat/remove-external-wallets

Conversation

@jjramirezn
Copy link
Copy Markdown
Contributor

@jjramirezn jjramirezn commented Apr 30, 2025

Very comprehensive PR removing external wallets logic from the app.

Flows tested: Send Link creation and claming (also guest and xchain flow), request creation and fulfillment (also with guest and xchain flow)

Right now is set merge to feat/index-claim-links, this is because it was based on that branch. Dont merge when approved

Summary by CodeRabbit

  • New Features

    • Cached gas fee estimation added for improved transaction feedback.
  • Removals

    • Multi-wallet support, wallet selection, and related UI components removed.
    • Wallet details page, wallet header, wallet card, and wallet toggle button components deleted.
    • External wallet support and associated flows eliminated.
    • "Wallet" section removed from app navigation.
    • Complex link creation, cross-chain, and gasless transaction flows removed.
    • Legacy wallet connection hooks and Redux wallet state management for multiple wallets deleted.
  • Improvements

    • Wallet management streamlined to a single wallet type with simplified connection and balance handling.
    • Wallet balance display and refresh made more direct and reliable.
    • Token and chain selectors now receive balances as props and fetch details asynchronously.
    • Payment and claim flows simplified, reducing complexity and potential user errors.
    • Wallet connection and modal visibility managed via Redux state.
    • Integration of Wagmi hooks alongside ZeroDev improves connection and transaction consistency.
    • UI components like FlowHeader simplified with fewer props and external dependencies.
  • Bug Fixes

    • Prevented errors from undefined wallet balances by defaulting to zero.
  • Refactor

    • Internal wallet logic and Redux state significantly simplified for better maintainability.
    • Removed legacy wallet and external wallet flags from components and hooks.
    • Claim and payment flows refactored to unify connection state and streamline transaction handling.
    • Utility functions updated to accurately identify supported tokens.
  • Chores

    • Removed unused dependencies and utility functions.
    • Cleaned up and reordered imports for clarity.
    • Added documentation comments to claim API method.

Loading
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