Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test mode: add a test parameter to the request #3

Closed
DanReyLop opened this issue Mar 6, 2019 · 3 comments · Fixed by #238
Closed

Test mode: add a test parameter to the request #3

DanReyLop opened this issue Mar 6, 2019 · 3 comments · Fixed by #238
Assignees
@reykjalin
Labels
priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. status: has pr The issue has a PR.

Comments

@DanReyLop
Copy link
Contributor

DanReyLop commented Mar 6, 2019 •
edited by LevinMedia
Loading

We'll need this, so better add it sooner rather than later. In the payment's Settings screen, we'll need a toggle to enable "test mode". At the very least, sandbox mode must:

  • Never make actual payments (maybe use the partner's sandbox API instead, if possible)
  • Work on non-HTTPS sites (the real gateway should only work on HTTPS)
@DanReyLop DanReyLop self-assigned this Mar 10, 2019
@allendav allendav added this to the V0.7 - Settings, On-Boarding / Account Creation milestone Jun 28, 2019
@allendav
Copy link
Contributor

allendav commented Jun 28, 2019 •
edited
Loading

We'll start with everything in test/sandbox mode. In milestone V0.7 we'll introduce for-reals accounts and need to be able to toggle between test and live

@allendav
Copy link
Contributor

allendav commented Sep 23, 2019 •
edited by reykjalin
Loading

Similar to what needs to happen on plugin activation, when the user toggles the plugin into (or out of) test mode, we'll need to ping the accounts endpoint on WPCOM to see if a test (or live) account needs setup. See also

14-gh-woocommerce-payments-server
and
12-gh-woocommerce-payments-server
and
#203

@allendav allendav added the priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. label Sep 23, 2019
@allendav allendav assigned reykjalin and unassigned DanReyLop Sep 23, 2019
@LevinMedia LevinMedia changed the title Add a plugin-wide "sandbox" mode Add a plugin-wide test mode Oct 5, 2019
@LevinMedia
Copy link
Contributor

Test mode checkbox is in the settings screen ready to go. Just need to make it functional.

@reykjalin reykjalin mentioned this issue Oct 15, 2019
Merged
@marcinbot marcinbot changed the title Add a plugin-wide test mode Test mode: add a test parameter to the request Oct 23, 2019
@reykjalin reykjalin added the status: has pr The issue has a PR. label Oct 25, 2019
@frosso frosso mentioned this issue Jun 3, 2021
Closed
@jessepearson jessepearson mentioned this issue Jun 7, 2021
Closed
@oaratovskyi oaratovskyi mentioned this issue Oct 7, 2021
Merged
4 tasks
@leonardola leonardola mentioned this issue Feb 13, 2023
Merged
5 tasks
@csmcneill csmcneill mentioned this issue Feb 25, 2023
Closed
@francoishvz francoishvz mentioned this issue Jul 3, 2023
Closed
@htdat htdat mentioned this issue Nov 3, 2023
Closed
@frosso frosso mentioned this issue Nov 8, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reykjalin reykjalin

Labels
priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. status: has pr The issue has a PR.
Projects
None yet
Milestone
No milestone
4 participants
@allendav @DanReyLop @LevinMedia @reykjalin