Skip to content

Update: Modify PIX Payments CTA title and change link opening behavior#1416

Merged
jjramirezn merged 2 commits intopeanut-walletfrom
fix/update-pix-cta-text
Nov 6, 2025
Merged

Update: Modify PIX Payments CTA title and change link opening behavior#1416
jjramirezn merged 2 commits intopeanut-walletfrom
fix/update-pix-cta-text

Conversation

@Zishan-7
Copy link
Contributor

@Zishan-7 Zishan-7 commented Nov 6, 2025

image

@vercel
Copy link

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
peanut-wallet Ready Ready Preview Comment Nov 6, 2025 9:08pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 6, 2025

Walkthrough

Modified the PIX Payments CTA in the home carousel: updated its display title and changed the click behavior to open the link in a new browser tab instead of using in-app routing.

Changes

Cohort / File(s) Change Summary
Home Carousel CTA
src/hooks/useHomeCarouselCTAs.tsx
Updated CTA title for merchant-map-pix from "20% Off with PIX Payments" to "Up to 10% cashback for Tier 2 users with PIX Payments" and changed onClick from router.push(URL) to window.open(URL, '_blank', 'noopener,noreferrer') to open externally in a new tab.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify the updated title copy matches product/marketing requirements.
  • Confirm new external open behavior is intended and does not break analytics or expected navigation flow.

Possibly related PRs

Suggested reviewers

  • Hugo0

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the two main changes: updating the PIX Payments CTA title and changing the link opening behavior from in-app navigation to new tab.
Description check ✅ Passed The description includes a screenshot/image preview of the changes, which is related to the PR objectives but lacks detailed text explanation of what was changed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/update-pix-cta-text

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1564746 and 8eb0803.

📒 Files selected for processing (1)
  • src/hooks/useHomeCarouselCTAs.tsx (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/hooks/useHomeCarouselCTAs.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Deploy-Preview

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai bot added the enhancement New feature or request label Nov 6, 2025
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7307b36 and 1564746.

📒 Files selected for processing (1)
  • src/hooks/useHomeCarouselCTAs.tsx (1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📚 Learning: 2024-10-23T09:38:27.670Z
Learnt from: jjramirezn
Repo: peanutprotocol/peanut-ui PR: 469
File: src/app/request/pay/page.tsx:32-64
Timestamp: 2024-10-23T09:38:27.670Z
Learning: In `src/app/request/pay/page.tsx`, if `linkRes` is not OK in the `generateMetadata` function, the desired behavior is to use the standard title and preview image without throwing an error.

Applied to files:

  • src/hooks/useHomeCarouselCTAs.tsx
📚 Learning: 2025-08-12T17:44:04.268Z
Learnt from: Zishan-7
Repo: peanutprotocol/peanut-ui PR: 1089
File: src/components/LandingPage/dropLink.tsx:35-42
Timestamp: 2025-08-12T17:44:04.268Z
Learning: In the Peanut UI project, opening the `/setup` route in a new tab from landing page CTAs is intentional design behavior to keep users on the marketing page while they start the setup process.

Applied to files:

  • src/hooks/useHomeCarouselCTAs.tsx
📚 Learning: 2025-09-08T03:13:09.111Z
Learnt from: jjramirezn
Repo: peanutprotocol/peanut-ui PR: 1190
File: src/app/(mobile-ui)/qr-pay/page.tsx:156-176
Timestamp: 2025-09-08T03:13:09.111Z
Learning: In the peanut-ui mobile app, the `/qr-pay` route is only accessed through the DirectSendQR component which always includes the qrCode parameter in the URL when redirecting users to the QR pay page after scanning MERCADO_PAGO or PIX QR codes.

Applied to files:

  • src/hooks/useHomeCarouselCTAs.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Deploy-Preview
🔇 Additional comments (1)
src/hooks/useHomeCarouselCTAs.tsx (1)

39-39: LGTM! Title update is clear and specific.

The updated title provides clearer information about the cashback offer and target user tier.

@jjramirezn jjramirezn merged commit f0612ce into peanut-wallet Nov 6, 2025
4 of 5 checks passed
@jjramirezn jjramirezn deleted the fix/update-pix-cta-text branch November 6, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants