Page MenuHomePhabricator

Remove WePay support from Phortune, and Restful/Httpful dependencies
ClosedPublic

Authored by epriestley on May 13 2019, 7:06 PM.
Tags
None
Referenced Files
F15411317: D20521.diff
Wed, Mar 19, 8:59 AM
F15411262: D20521.diff
Wed, Mar 19, 8:57 AM
F15411202: D20521.diff
Wed, Mar 19, 8:54 AM
F15411123: D20521.diff
Wed, Mar 19, 8:51 AM
F15410998: D20521.diff
Wed, Mar 19, 8:46 AM
F15410924: D20521.diff
Wed, Mar 19, 8:43 AM
F15410131: D20521.diff
Wed, Mar 19, 6:48 AM
F15390682: D20521.id.diff
Sat, Mar 15, 6:26 AM
Subscribers

Details

Reviewers
amckinley
Summary

Ref PHI1166. I'm documenting our dependencies, and we have approximately 5,000 lines of external code to support WePay as a Phortune provider. We don't use it, I'm almost certain it doesn't work, and we have no plans to use it in the near future. If we did pursue it, I'd probably just wrap the API in a 100-line WePayFuture anyway since 5K lines of dependencies to make a couple method calls is ridiculous.

Test Plan

Grepped for wepay, httpful, restful.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable