Skip to content

Conversation

@rafaeluphold
Copy link

@rafaeluphold rafaeluphold commented Oct 28, 2025

Description

Add PayPal and Venmo testing accounts to the environments page.

Related issues

SWY-3438

Screenshots

Screenshot 2025-10-29 at 11 08 39

@rafaeluphold rafaeluphold self-assigned this Oct 28, 2025
Copilot AI review requested due to automatic review settings October 28, 2025 18:47
@rafaeluphold rafaeluphold requested a review from a team as a code owner October 28, 2025 18:47
@github-actions
Copy link

github-actions bot commented Oct 28, 2025

🚀 Preview has been deployed.

Name Link
🔨 Commit SHA 6677f77
🔍 Deploy log https://github.com/uphold/topper-docs/actions/runs/18905958618
🚀 Preview URL https://2ad40e67.topper-docs.pages.dev

Copy link

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 adds documentation for PayPal and Venmo test accounts in the sandbox environment. It provides testing credentials and usage guidelines to help developers test payment integrations.

  • Adds test account credentials for PayPal and Venmo sandbox testing
  • Documents country restrictions (US-only for both payment methods)
  • Includes usage guidelines and cautions to prevent shared test account issues

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

@rafaeluphold rafaeluphold force-pushed the feature/paypal-venmo-accounts branch from 645b3a9 to b209237 Compare October 29, 2025 11:09
@rafaeluphold rafaeluphold requested a review from Copilot October 29, 2025 11:09
Copy link

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 1 out of 1 changed files in this pull request and generated 2 comments.


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

@rafaeluphold rafaeluphold force-pushed the feature/paypal-venmo-accounts branch from b209237 to e38f8ff Compare October 29, 2025 11:13
@rafaeluphold rafaeluphold requested a review from Copilot October 29, 2025 11:14
Copy link

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 1 out of 1 changed files in this pull request and generated 1 comment.


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

:::note
1. Both PayPal and Venmo payment methods are only available for US users.

2. The accounts provided will produce the success outcome. To trigger the `name-mismatch` outcome, use a user where the email has "+lowscore" appended (e.g., `[email protected]`).
Copy link

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

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

The documentation mentions triggering a name-mismatch outcome by appending '+lowscore' to the email, but it's unclear whether this applies to both PayPal and Venmo or only PayPal. Since Venmo uses usernames rather than emails, clarify if this behavior applies to both payment methods and provide an example for Venmo if applicable.

Suggested change
2. The accounts provided will produce the success outcome. To trigger the `name-mismatch` outcome, use a user where the email has "+lowscore" appended (e.g., `[email protected]`).
2. The accounts provided will produce the success outcome.
- **PayPal:** To trigger the `name-mismatch` outcome, use a user where the email has "+lowscore" appended (e.g., `[email protected]`).
- **Venmo:** The `name-mismatch` outcome cannot be triggered using the provided Venmo test accounts, as Venmo uses usernames rather than emails and does not support the "+lowscore" mechanism.

Copilot uses AI. Check for mistakes.
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