Comments

czigor created an issue. See original summary.

czigor’s picture

Status: Active » Needs review
StatusFileSize
new28.04 KB

This patch contains the patch in #2331119-39: PayPal support, so we need to remove it from here once it gets committed.

Status: Needs review » Needs work

The last submitted patch, 2: commerce_braintree-paypal_credit-2916512-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

czigor’s picture

Status: Needs work » Needs review
StatusFileSize
new28.78 KB

Adding the paypal_credit configuration to the schema.

czigor’s picture

Implementing paypal credit as a new PaymentMethodType instead. The patch still contains the patch in #2331119-39: PayPal support too.

czigor’s picture

Some additional fixes. (Interdiff is just a diff.)

czigor’s picture

StatusFileSize
new0 bytes

Reroll after #2331119: PayPal support got committed.

czigor’s picture

0 bytes has advantages too, but this is better.

czigor’s picture

The last submitted patch, 7: commerce_braintree-paypal_credit-2916512-7.patch, failed testing. View results

Status: Needs review » Needs work

The last submitted patch, 8: commerce_braintree-paypal_credit-2916512-8.patch, failed testing. View results

czigor’s picture

StatusFileSize
new6.73 KB

Fixing some variable naming and adding some code comments.

czigor’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 12: commerce_braintree-paypal_credit-2916512-12.patch, failed testing. View results

czigor’s picture

  • mglaman committed 061e53b on 8.x-1.x authored by czigor
    Issue #2916512 by czigor: Paypal credit support
    
mglaman’s picture

Status: Needs review » Fixed

\o/ yay, committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.