Add a payment authorized payment status that we can use for payments for which no money has been transferred yet, but where we know that we can make the transfer unconditionally at any time.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | payment_2177133_6.patch | 934 bytes | xano |
Comments
Comment #1
xanoComment #2
sigveio commentedSorry, I see you asked me to review this. In what sense / to what extent did you want me to review it please?
(I am not much into D8 yet, and do not have a store setup or payment methods ready for it, so with such a simple/small patch I can't really add anything else than to tell you whether or not the id/label/description seems sensible. Is that perhaps what you were after?)
Comment #3
xanoThis issue was a response to one of your earlier requests, so I was hoping to hear whether this solution will address your needs, or if it doesn't.
Comment #4
sigveio commentedI see. Well - the payment status in itself certainly looks to be what I was looking for... so it seems fine to me.
Next up we'd need the new Authorized state to be implemented in "context" modules such as payment_ubercart. They should move the order past checkout not only on success, but also on the authorized status.
(And then extend payment statuses with the option of adding payee/payer messages like we discussed earlier - but that is probably a topic for a separate issue.)
Comment #5
xano1: payment_2177133_1.patch queued for re-testing.
Comment #6
xanoThis patch uses a description that should communicate the need for action a bit better.
Comment #7
xano6: payment_2177133_6.patch queued for re-testing.
Comment #8
xano