697805 - Allow developers to test their paypal setup works

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P3)

Description

[Andy McKay]

There's a couple of restrictions on using chained payments accounts. We can't test for those programmatically, but we can send a paykey request to the account and it will fail if they can't. This involves send no money and should be safe to do as much as we want. According to Brandee this will:

"This will work to verify that an account has been created(confirmed email address) and is able to send money (has a funding source.)  The only thing it will not tell you is if the Primary can accept Guest Payments (credit card payments without PayPal login).  Only the sender would see (or not see) that in the payments screen."

Comment 1

[Wil Clouser [:clouserw]]

Handing this off to gkoberger, but ask andym if it's not clear.  I'm picturing this happening ajax-ily on your edit payments page just as thumbs up that stuff is working right.

Comment 2

[Gregory Koberger (:gkoberger)]

According to Andy, we need to do this:

    paypal.get_paykey(dict(uuid=somerandomhash, slug='foo', amount=1.00, memo='testing', email=addon.paypal_id, pattern='apps.purchase.finished', chains=settings.PAYPAL_CHAINS))

(See https://github.com/mozilla/zamboni/blob/master/apps/addons/views.py#L506)

Just posting this here in case I get hit by a bus and someone else needs to know how to finish it :)

Comment 3

[Gregory Koberger (:gkoberger)]

1) Filed bug 708084
2) Code: https://github.com/mozilla/zamboni/commit/a5fded6a

Comment 4

[Gregory Koberger (:gkoberger)]

Bug 708084 was closed, so I can close this now that I've been able to verify it actually works on preview. I also added tests and fixed a few things:

https://github.com/mozilla/zamboni/compare/52d12c...8a43f62

Andy wants a "Check PayPal" button, which we can add if we did it. However, it will check on page load and on every change (when the user moves off the field, not on every keystroke), so I think that should be enough.