Skip to content

Conversation

@felicio
Copy link
Collaborator

@felicio felicio commented Jul 10, 2025

No description provided.

Copilot AI review requested due to automatic review settings July 10, 2025 10:30
@github-project-automation github-project-automation bot moved this to 🚧 In Progress in Web & User Interfaces Jul 10, 2025
@changeset-bot
Copy link

changeset-bot bot commented Jul 10, 2025

🦋 Changeset detected

Latest commit: 2eee0c0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@status-im/wallet Patch
portfolio Patch
wallet Patch
api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Jul 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
status-api ❌ Failed (Inspect) Jul 10, 2025 10:51am
status-portfolio ❌ Failed (Inspect) Jul 10, 2025 10:51am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
status-components ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2025 10:51am

@status-im-auto
Copy link
Member

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ bc7ddb2 #1 2025-07-10 10:32:02 ~52 sec wallet 📦zip

Copy link
Contributor

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 centralizes the “Buy crypto” UI by moving the drawer component into the shared wallet package and updating all consumers to use it.

  • Adds BuyCryptoDrawer (and related types) to packages/wallet
  • Implements the new BuyCryptoDrawer component and exports it
  • Refactors two apps to remove duplicate buy-drawer logic and wrap their UIs with the shared component

Reviewed Changes

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

File Description
packages/wallet/src/components/index.tsx Exports the new BuyCryptoDrawer and its types
packages/wallet/src/components/buy-crypto-drawer/index.tsx Introduces the standalone BuyCryptoDrawer component
apps/wallet/src/routes/portfolio/assets/-components/token.tsx Replaces inline buy buttons with BuyCryptoDrawer
apps/portfolio/src/app/_components/buy-crypto-drawer.tsx Wraps children in shared BuyCryptoDrawerBase

} else {
const providerUrls = {
mercuryo: `https://exchange.mercuryo.io/?type=buy&network=${network.id}&currency=${currency.code}&address=${account.address}&hide_address=false&fix_address=true&widget_id=6a7eb330-2b09-49b7-8fd3-1c77cfb6cd47`,
moonpay: `https://buy.moonpay.com?apiKey=pk_live_YQC6CQPA5qqDu0unEwHJyAYQyeIqFGR`,
Copy link

Copilot AI Jul 10, 2025

Choose a reason for hiding this comment

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

The Moonpay URL is hardcoded without any dynamic query parameters. To ensure the correct network, currency, and recipient, include network=${network.id}, currency=${currency.code}, and address=${account.address} in the URL template.

Suggested change
moonpay: `https://buy.moonpay.com?apiKey=pk_live_YQC6CQPA5qqDu0unEwHJyAYQyeIqFGR`,
moonpay: `https://buy.moonpay.com?apiKey=pk_live_YQC6CQPA5qqDu0unEwHJyAYQyeIqFGR&network=${network.id}&currency=${currency.code}&address=${account.address}`,

Copilot uses AI. Check for mistakes.
@felicio felicio merged commit 264da50 into main Jul 10, 2025
7 of 10 checks passed
@felicio felicio deleted the felicio/m/buy branch July 10, 2025 10:51
@github-project-automation github-project-automation bot moved this from 🚧 In Progress to Done in Web & User Interfaces Jul 10, 2025
@felicio felicio self-assigned this Jul 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants