Differential D10670

Implement Phortune charge updates
ClosedPublic
Actions

Authored by epriestley on Oct 9 2014, 4:59 PM.

Tags

None

Referenced Files

	F13153098: D10670.diff
	Sun, May 5, 12:32 AM

	Unknown Object (File)
	Sun, Apr 28, 6:06 AM

	Unknown Object (File)
	Wed, Apr 24, 11:30 PM

	Unknown Object (File)
	Thu, Apr 11, 7:55 AM

	Unknown Object (File)
	Tue, Apr 9, 5:42 AM

	Unknown Object (File)
	Apr 1 2024, 11:49 AM

	Unknown Object (File)
	Mar 31 2024, 4:56 PM

	Unknown Object (File)
	Mar 27 2024, 3:03 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T2787: Build Phortune into a real accounts/billing application

Commits

Restricted Diffusion Commit
rPf41ae2228ae0: Implement Phortune charge updates

Summary

Ref T2787. These don't necessarily do a ton yet, but we can get PayPal out of hold, at least.

Test Plan

Updated charges from all providers. Cleared a PayPal hold.

Diff Detail

Repository

Branch

fund24

Lint

Lint Passed

Severity	Location	Code	Message
Advice	src/applications/fund/phortune/FundBackerProduct.php:82	XHP16	TODO Comment
Advice	src/applications/phortune/provider/PhortuneBalancedPaymentProvider.php:221	XHP16	TODO Comment
Advice	src/applications/phortune/provider/PhortunePayPalPaymentProvider.php:222	XHP16	TODO Comment
Advice	src/applications/phortune/provider/PhortunePayPalPaymentProvider.php:226	XHP16	TODO Comment
Advice	src/applications/phortune/provider/PhortuneStripePaymentProvider.php:202	XHP16	TODO Comment
Advice	src/applications/phortune/provider/PhortuneWePayPaymentProvider.php:212	XHP16	TODO Comment

Unit

No Test Coverage

Build Status

Buildable 2790
Build 2794: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 25626.Oct 9 2014, 4:59 PM

epriestley retitled this revision from to Implement Phortune charge updates.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T2787: Build Phortune into a real accounts/billing application.

Herald added a subscriber: epriestley. · View Herald TranscriptOct 9 2014, 4:59 PM

Harbormaster completed remote builds in B2790: Diff 25626.Oct 9 2014, 5:00 PM

btrahan accepted this revision.Oct 9 2014, 11:17 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Oct 9 2014, 11:17 PM

Closed by commit rPf41ae2228ae0: Implement Phortune charge updates (authored by epriestley, committed by epriestley).Oct 9 2014, 11:57 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

fund/

phortune/

FundBackerProduct.php

2 lines

phortune/

controller/

PhortuneAccountViewController.php

13 lines

PhortuneCartUpdateController.php

36 lines

PhortuneCartViewController.php

3 lines

provider/

PhortuneBalancedPaymentProvider.php

48 lines

PhortunePayPalPaymentProvider.php

57 lines

PhortunePaymentProvider.php

4 lines

PhortuneStripePaymentProvider.php

36 lines

PhortuneTestPaymentProvider.php

4 lines

PhortuneWePayPaymentProvider.php

57 lines

storage/

PhortuneCart.php

18 lines

PhortuneCharge.php

4 lines

Diff 25626

src/applications/fund/phortune/FundBackerProduct.php

Loading...

src/applications/phortune/controller/PhortuneAccountViewController.php

Loading...

src/applications/phortune/controller/PhortuneCartUpdateController.php

Loading...

src/applications/phortune/controller/PhortuneCartViewController.php

Loading...

src/applications/phortune/provider/PhortuneBalancedPaymentProvider.php

Loading...

src/applications/phortune/provider/PhortunePayPalPaymentProvider.php

Loading...

src/applications/phortune/provider/PhortunePaymentProvider.php

Loading...

src/applications/phortune/provider/PhortuneStripePaymentProvider.php

Loading...

src/applications/phortune/provider/PhortuneTestPaymentProvider.php

Loading...

src/applications/phortune/provider/PhortuneWePayPaymentProvider.php

Loading...

src/applications/phortune/storage/PhortuneCart.php

Loading...

src/applications/phortune/storage/PhortuneCharge.php

Loading...