Tryton - Issues

 

Issue7760

Title More robusteness when capturing
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type feature request Components account_payment_stripe
Assigned To ced Keywords review
Reviews 47711002
View: 47711002

Created on 2018-10-03.15:00:01 by ced, last changed by roundup-bot.

Messages
New changeset c4ce9eeb39e9 by Cédric Krier in branch 'default':
Use queue to charge and capture
https://hg.tryton.org/tryton-env/rev/c4ce9eeb39e9
New changeset c236dc6b8a1d by Cédric Krier in branch 'default':
Use queue to charge and capture
https://hg.tryton.org/modules/account_payment_stripe/rev/c236dc6b8a1d
review47711002 updated at https://codereview.tryton.org/47711002/#ps20001
review47711002 updated at https://codereview.tryton.org/47711002/#ps1
msg44296 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-10-03.15:00:00
The capturing is done with a button but this button has a commit inside to ensure to not loose response from Stripe. But this makes the button/method not usable inside a larger process.
I think the process should be isolated like the charging and executed in a separate transaction (with cron).
So I propose to define cron task and also to use queue to process ASAP.
History
Date User Action Args
2018-10-21 21:34:44roundup-botsetmessages: + msg44506
2018-10-21 21:34:33roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg44505
2018-10-03 15:54:16reviewbotsetmessages: + msg44299
2018-10-03 15:26:42reviewbotsetnosy: + reviewbot
messages: + msg44297
2018-10-03 15:17:18cedsetstatus: in-progress -> testing
reviews: 47711002
keyword: + review
2018-10-03 15:00:01cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)