Describe your bug or feature request.

The order payments list cannot easily be customized as it is... Just like we did for other Commerce entity types, let's replace the payment list builder by a view.

Note that we need a new views are handler for the payment total summary, patch to follow.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jsacksick created an issue. See original summary.

jsacksick’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
173.94 KB
30.78 KB

One thing I wasn't really sure about while doing the conversion was where to output the AVS response.

Right now, it's displayed right before the "operations", in a column with no table header, which kind of looks weird... Not sure if that is intentional or if that is a mistake, we probably should output it in the "Payment" column I suppose?

Also, we probably should output the Refunded amount on a new line.

This is how it looks with the view, and minor changes now:

Let's see if the tests are green!

Status: Needs review » Needs work

The last submitted patch, 2: 3302853-2.patch, failed testing. View results

jsacksick’s picture

Status: Needs work » Needs review
FileSize
31.78 KB

  • jsacksick committed 0358a61 on 8.x-2.x
    Issue #3302853 by jsacksick, rinasek: Replace the PaymentListBuilder by...
jsacksick’s picture

Status: Needs review » Fixed

Committed!

  • jsacksick committed 9dfab0b on 3.0.x
    Issue #3302853 by jsacksick, rinasek: Replace the PaymentListBuilder by...

Status: Fixed » Closed (fixed)

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