Skip to content

Comments

Issue #2804227 Add $order->getBalance()#786

Closed
jackbravo wants to merge 18 commits intodrupalcommerce:8.x-2.xfrom
jackbravo:2804227-get-order-balance-jack
Closed

Issue #2804227 Add $order->getBalance()#786
jackbravo wants to merge 18 commits intodrupalcommerce:8.x-2.xfrom
jackbravo:2804227-get-order-balance-jack

Conversation

@jackbravo
Copy link
Contributor

This is a rebase of PR 508. Adding a commerce_order_post_update_6() function.

@jackbravo
Copy link
Contributor Author

I added one more change. Now when we calculate the balance, we consider the payment status. The tests are also updated for this. So I think this is now RTBC, but will wait for someone else to review it.

@steveoliver
Copy link
Contributor

Looking good! Needs a rebase.

@steveoliver
Copy link
Contributor

I rebased this branch and put it up for testing at #821. I feel like I may have munged up a commit or two, so let's see..

@bojanz
Copy link
Contributor

bojanz commented Aug 16, 2018

Continued on the issue https://www.drupal.org/node/2804227

@bojanz bojanz closed this Aug 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants