7/13/2015 8:53:23 PM
acoording to the description in the bug
Fix typo in PaymentStateMachineHelper (preventing the transaition from PENDING to SUCCESS)
Handle UNKNOWN cases rhe same way everywhere (where we just simply fetch latest state from plugin but do not re-attenmpt a payment transaction)
Add Payment tests around paymentTransactionExternalKey and logic around UNKNWON use cases
|
7/13/2015 7:44:57 AM
commit also simplifies the account update path,
which used to rely on dummy default values. We now simply
rely on null values to keep them as-is.
See https://github.com/killbill/killbill/wiki/Kill-Bill-Migration-0.14.x-to-0.16.x
for the upgrade DDL.
Signed-off-by: Pierre-Alexandre Meyer <pierre@mouraf.org>
|
|
|
|
|
|
|
7/10/2015 2:42:27 PM
properties), and add a test case
|
|
|
7/10/2015 2:09:42 PM
bus events
|
7/10/2015 1:59:45 AM
test pass.
Need a second code review pass, and iron details about what should be a payment config property, versus hard-coded value
|
7/9/2015 4:02:28 PM
(See re-opened #342)
Will address the rework on IncompletePaymentTransactionTask based on events and notificationQ in a subsequent commit
|