Skip to content

refac: reorganize wallet connection architecture#182

Merged
akiraonstarknet merged 1 commit intoakiraonstarknet:devfrom
EjembiEmmanuel:akira-dev
Oct 10, 2025
Merged

refac: reorganize wallet connection architecture#182
akiraonstarknet merged 1 commit intoakiraonstarknet:devfrom
EjembiEmmanuel:akira-dev

Conversation

@EjembiEmmanuel
Copy link

PR Fixes:

  • Refactors wallet connection architecture for better modularity and maintainability
  • Consolidates wallet logic into dedicated components
  • Removes legacy hook-based wallet implementation

What Changed

This PR reorganizes the wallet connection architecture by:

  • Created ConnectButton component: Extracted wallet connection UI logic into a dedicated, reusable component
  • Moved Providers to components: Relocated from src/app/template.tsx to src/components/Providers.tsx for better organization
  • Added connectors module: Centralized Starknet wallet connector configuration (ArgentMobile, Braavos, ArgentX, etc.)
  • Removed legacy hooks: Deleted useWalletConnection.tsx and wallet.ts in favor of the new architecture
  • Updated references: Updated Navbar.tsx, Strategy.tsx, YieldCard.tsx, and layout.tsx to use the new components

Benefits

  • ✨ Cleaner separation of concerns
  • 🔧 Easier to maintain and test wallet functionality
  • 📦 Better component reusability

Vercel website link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue
  • My PR passes all checks (build, lint, formatting, etc)

@vercel
Copy link

vercel bot commented Oct 10, 2025

@EjembiEmmanuel is attempting to deploy a commit to the zkdice-xyz's projects Team on Vercel.

A member of the Team first needs to authorize it.

- Extract Providers to separate component
- Create dedicated ConnectButton component
- Add connectors module
- Remove legacy wallet hooks
@akiraonstarknet akiraonstarknet merged commit e33f74f into akiraonstarknet:dev Oct 10, 2025
0 of 3 checks passed
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