Skip to content

[Campaign Launcher Client] Replace custom wallet modal with Reown AppKit#848

Open
flopez7 wants to merge 2 commits intodevelopfrom
flopez/reown
Open

[Campaign Launcher Client] Replace custom wallet modal with Reown AppKit#848
flopez7 wants to merge 2 commits intodevelopfrom
flopez/reown

Conversation

@flopez7
Copy link
Copy Markdown
Collaborator

@flopez7 flopez7 commented Apr 8, 2026

Issue tracking

Freestyle

Context behind the change

The previous custom wallet modal was limiting the connection UX and required us to maintain wallet-specific UI logic ourselves. Replaced it with Reown AppKit so users get a more standard wallet-selection flow while we keep the existing wagmi-based code unchanged.

How has this been tested?

Ran dashboard locally.

Release plan

Add new VITE_APP_TERMS_URL and VITE_APP_PRIVACY_URL with terms and privacy policy URLs

Potential risks; What to monitor; Rollback plan

None

Copilot AI review requested due to automatic review settings April 8, 2026 16:44
@flopez7 flopez7 changed the title replace custom wallet modal with Reown AppKit [Campaign Launcher Client] Replace custom wallet modal with Reown AppKit Apr 8, 2026
@flopez7 flopez7 self-assigned this Apr 8, 2026
@flopez7 flopez7 moved this to In review in HuFi 2026 (H1) Apr 8, 2026
@flopez7 flopez7 requested review from KirillKirill and dnechay April 8, 2026 16:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR replaces the in-app custom wallet selection UI with Reown AppKit while keeping the existing Wagmi-based connection logic, and adds a new legal URL env var for AppKit’s required links.

Changes:

  • Integrate Reown AppKit + Wagmi adapter and wrap the app in AppKitProvider.
  • Replace the custom wallet popover/modal with an AppKit “Connect” view trigger.
  • Update dependencies/lockfile and add VITE_APP_LEGAL_URL to .env.example.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
campaign-launcher/client/src/providers/WagmiProvider.tsx Introduces AppKit provider/config and swaps Wagmi config creation to AppKit’s wagmi adapter.
campaign-launcher/client/src/components/ConnectWallet/index.tsx Removes custom wallet picker UI and opens AppKit’s connect view; keeps mobile sign-in prompt behavior.
campaign-launcher/client/package.json Removes MetaMask/WalletConnect provider deps and adds Reown AppKit + @wagmi/core.
campaign-launcher/client/.env.example Adds VITE_APP_LEGAL_URL example value.
campaign-launcher/client/yarn.lock Lockfile updates for the dependency swap (Reown AppKit, WalletConnect version changes, removals).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread campaign-launcher/client/src/providers/WagmiProvider.tsx
Comment thread campaign-launcher/client/package.json
@flopez7 flopez7 marked this pull request as draft April 8, 2026 16:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread campaign-launcher/client/.env.example
@dnechay
Copy link
Copy Markdown
Contributor

dnechay commented Apr 9, 2026

We don't use reown because we can't customize connection popup to fit our design (esp. new one)
I'm not. agains using it, but would suggest that we get back to it once redesign is finished, unless @KirillKirill has some other info related to it

@flopez7
Copy link
Copy Markdown
Collaborator Author

flopez7 commented Apr 9, 2026

I started integrating it yesterday because it was requested, and after testing it I think it may be worth keeping. Our current modal only supports 3 wallets, while Reown supports many more out of the box, which is much better for users. Also, it's customizable, so we may be able to adapt it to the redesign instead of maintaining our own custom modal.
@dnechay @KirillKirill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

4 participants