Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Let @pjf do PayPal links #2177

Closed
pjf opened this issue Mar 24, 2014 · 21 comments
Closed

Let @pjf do PayPal links #2177

pjf opened this issue Mar 24, 2014 · 21 comments

Comments

@pjf
Copy link
Contributor

pjf commented Mar 24, 2014

We have a backlog of PayPal links that need to be made in the support queues. I'm happy to do these when I have spare cycles; but AFAIK I don't have the relevant permissions to do so yet.

How do I get about obtaining these?

Pinging @bruceadams on this at least.

@clone1018
Copy link
Contributor

We also need someone besides @whit537 to have Balanced access to resolve other issues.

@chadwhitacre
Copy link
Contributor

Balanced is easier thanks to balanced/balanced-dashboard#1052.

Coinbase is harder because they don't have user management (I've asked for it on Twitter), but I guess that's a separate issue.

@chadwhitacre
Copy link
Contributor

Actually, this is about PayPal, which should be a matter of setting is_admin for @pjf. And I did that a bit ago.

@chadwhitacre
Copy link
Contributor

@pjf You should have perms to modify PayPal status in the db. Have you and @bruceadams tried going through the process?

@bruceadams
Copy link
Contributor

Um, i do the PayPal setups via a script that is exercising direct database through Heroku (I think, I don't fully understand the plumbing).

@chadwhitacre
Copy link
Contributor

@bruceadams Right. Would you be willing/able to cross-train @pjf on that process?

@pjf
Copy link
Contributor Author

pjf commented Mar 25, 2014

So, I'm guessing I'm going to need a Heroku account to start with for this? ;)

And I can confirm I seem to have is_admin when using the Gittip website.

@chadwhitacre
Copy link
Contributor

Ah, yes. You do, so that you can grab the production DB url.

@chadwhitacre
Copy link
Contributor

@pjf Invite to Heroku sent.

@pjf
Copy link
Contributor Author

pjf commented Mar 25, 2014

@whit537 Invite to へろく accepted.

@chadwhitacre
Copy link
Contributor

💃

@pjf
Copy link
Contributor Author

pjf commented Mar 25, 2014

And invite to Balanced accepted as well.

@pjf
Copy link
Contributor Author

pjf commented Mar 25, 2014

💃 is one of my favourite web frameworks! :)

http://www.perldancer.org/

#OffTopic. ;)

@chadwhitacre
Copy link
Contributor

Labeling this TeamX ★. Let's get @pjf up to speed here!

@pjf
Copy link
Contributor Author

pjf commented Mar 28, 2014

So, just a heads up that I'm starting the two weeks of the year in which I make most of my income for the entire year. That's immediately followed by an unconference which is so insanely awesome that I don't expect to sleep for three days. Overall, this may seriously crunch my time for the next month.

Consequently, this is a heads up that I may be the bottleneck in this process. ;)

@chadwhitacre
Copy link
Contributor

What's our status here? I believe @patcon can also cross-train if necessary?

@pjf
Copy link
Contributor Author

pjf commented May 13, 2014

Right, I'm back on the support train. How are we doing paypal connects? Still with a script that frobnicates the database directly? :)

@seanlinsley
Copy link
Contributor

@pjf
Copy link
Contributor Author

pjf commented May 13, 2014

Oh noes! I'm new to heroku, and clearly missing something!

pjf@qapla $ heroku run --app gittip 'invoke set_paypal_email -u user -p [email protected] -a decafbad'
Running `invoke set_paypal_email -u user -p [email protected] -a decafbad` attached to terminal... up, run.2324
No idea what '-p' is!

There's another/different step involved here, isn't there? @seanlinsley 's link seems to imply a heroku prompt.... how do I get there?

patcon referenced this issue May 13, 2014
The usage docstring was wrong (no more -p, it's -e now). And also the
api key is actually required. The way these are implemented you have to
explicitly say "None" if you're cool with no API key.
@pjf
Copy link
Contributor Author

pjf commented May 13, 2014

So, I've got this working via heroku run --app gittip (without needing to try and build gittip locally), but the command syntax is totally different than what's in the help. I did a bit more faffing around, but it amounts to:

heroku run --app gittip invoke set_paypal_email --username=pjf --api-key-fragment=decafbad [email protected]

However set_paypal_email barfs when invoking load_prod_envvars(), as that would invoke heroku from heroku, causing the universe to implode. Commenting out that call results in things working fine.

Suggestions:

  • Update set_paypal_email to have an optional switch which (declines to) loads the production environment variables. Bonus points if it's able to sniff the environment and make the right choice automatically.
  • I can't seem to verify paypal addresses via the gittip website. But using psql to inspect the table directly shows that they're set.
  • Since this is such a common request, we really should let people set their own paypal details. There's a question of how to avoid fraud here, but our current processes don't necessarily protect against that, either. (I can make an account, ask for a paypal link while it's not getting funds (which doesn't look suspicious), and then once that's set up then proceed to fraud away by transferring from a stolen credit card.)

I can't seem to find a ticket for "let people do their own paypal setup", but if anyone points me at it, I'll gladly toss some bounty coins on it. This is by far one of our most common support requests.

@pjf
Copy link
Contributor Author

pjf commented May 13, 2014

I've broken all my suggestions out into separate issues, so closing this now.

@Changaco may rejoice. :D

@pjf pjf closed this as completed May 13, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants