impress-org / give-wepay Goto Github PK

Description

Currently, this addon does not deactivate automatically when Give code deactivates. Check stripe addon.

Problem: Users are complaining the plugin is incompatible with the latest WePay API and there's not a way to use the older version.

Overview

The new WePay API version (2017-02-01) has breaking changes that are resulting in errors on donations.

Steps to replicate

1. update the WePay API version on your test App in the wepay API (see notes belowe in the Other info section before doing this). In my staging account at stage.wepay.com/home there was an alert on the top right corner of the page telling me to update (see image below)
2. Process a donation.
3. You should get an error related to the postal code: address.postal_code parameter is required

Customer reported, I and @mathetos were able to replicate.

Other Info

The Developer docs don't yet have the latest version of the API documented: https://stage.wepay.com/developer/version however there was a link (that I charged right past) during the "ARE YOU SURE" warnings when I updated the API on my account that may link to changelogs for the latest. Though it looks like the December changelog has significant back-compat breakage too, so that might be enough.

When PHP 5.3 is in use there are "Can't use in write context" errors

When you change the status from "Complete" to "Refunded" there should be an option that appears which allows the donor to refund in the gateway similar to how Stripe works.

Referenced here: https://secure.helpscout.net/conversation/214474522/2455/?folderId=672194

Do we have CAD support, or plans to implement it?

user request: https://secure.helpscout.net/conversation/290164486/0/?folderId=672194

Their text:

I understand that WePay allow for donors to pay for the processing fee, but is that something that donor could choose (with a checkbox) to pay the fees or not? Right now, it seems to me that this option can only be set by the admin backend.

This would be a compelling, value-adding feature.

I noticed that when testing a payment I received:

Although, the payment had been "released" https://www.wepay.com/developer/reference/checkout#states

Which we are not accounting for this status.

Shorter CC checkout form.

The field should be conditional based on whether the gateway is using the onsite or offsite integration.

Reported here: https://secure.helpscout.net/conversation/203132261/2085/?folderId=672200

We did a lot of improvement to the Stripe preapproval integration. The same should be done for WePay. This includes:

• Better admin UI in the backend
• A notification email to the admin and donor

The rest of the addons and core are moving away from checkboxes and to "enabled/disabled" radio options.

I was just testing an issue for a customer, and noticed that my sandbox is automatically set to preapprove only for WePay. I suspect this is due to the change in 1.8 with regard to checkboxes, but didn't spend much time testing.