Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(427)

Issue 49391002: sale: Use task queue to process sale (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 months ago by ced
Modified:
1 month, 3 weeks ago
Reviewers:
pokoli, rietveld-bot, reviewbot
Visibility:
Public.

Description

It adds configured delay for processing sale which provides a grace period. And it allows to reset to draft confirmed but not processed sale. issue7460

Patch Set 1 #

Patch Set 2 : Remove unused user #

Total comments: 2

Patch Set 3 : Update to tip #

Patch Set 4 : Update to tip #

Patch Set 5 : Update to tip #

Unified diffs Side-by-side diffs Delta from patch set Stats (+82 lines, -137 lines) Patch
M CHANGELOG View 1 2 3 1 chunk +2 lines, -0 lines 0 comments Download
M configuration.py View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
M invoice.py View 1 2 3 2 chunks +7 lines, -9 lines 0 comments Download
M sale.py View 1 2 3 6 chunks +12 lines, -10 lines 0 comments Download
M sale.xml View 1 1 chunk +0 lines, -22 lines 0 comments Download
M stock.py View 1 2 3 7 chunks +54 lines, -80 lines 0 comments Download
M tests/scenario_sale.rst View 1 2 3 11 chunks +0 lines, -13 lines 0 comments Download
M tests/scenario_sale_empty.rst View 1 2 3 1 chunk +0 lines, -3 lines 0 comments Download
M view/configuration_form.xml View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 16
ced
4 months ago (2018-05-24 12:53:16 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/49391002
4 months ago (2018-05-24 12:57:40 UTC) #2
ced
Remove unused user
4 months ago (2018-05-24 13:12:21 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/49391002
4 months ago (2018-05-24 13:31:34 UTC) #4
pokoli
https://codereview.tryton.org/49391002/diff/10011/sale.py File sale.py (right): https://codereview.tryton.org/49391002/diff/10011/sale.py#newcode276 sale.py:276: 'process': { shouldn't we remove the process button? For ...
4 months ago (2018-05-24 14:14:45 UTC) #5
ced
https://codereview.tryton.org/49391002/diff/10011/sale.py File sale.py (right): https://codereview.tryton.org/49391002/diff/10011/sale.py#newcode276 sale.py:276: 'process': { On 2018/05/24 14:14:45, pokoli wrote: > shouldn't ...
4 months ago (2018-05-24 14:29:54 UTC) #6
ced
Update to tip
2 months, 3 weeks ago (2018-06-26 11:26:00 UTC) #7
reviewbot
flake8 OK URL: https://codereview.tryton.org/49391002
2 months, 3 weeks ago (2018-06-26 11:32:37 UTC) #8
reviewbot
patch is not applicable URL: https://codereview.tryton.org/49391002
2 months, 3 weeks ago (2018-06-26 20:56:01 UTC) #9
reviewbot
patch is not applicable URL: https://codereview.tryton.org/49391002
2 months, 3 weeks ago (2018-06-26 20:58:14 UTC) #10
reviewbot
patch is not applicable URL: https://codereview.tryton.org/49391002
2 months, 3 weeks ago (2018-06-26 21:06:04 UTC) #11
ced
Update to tip
2 months, 3 weeks ago (2018-06-26 21:10:31 UTC) #12
reviewbot
flake8 OK URL: https://codereview.tryton.org/49391002
2 months, 3 weeks ago (2018-06-26 21:27:41 UTC) #13
ced
Update to tip
2 months, 1 week ago (2018-07-10 15:27:57 UTC) #14
reviewbot
flake8 OK URL: https://codereview.tryton.org/49391002
2 months, 1 week ago (2018-07-10 15:41:46 UTC) #15
rietveld-bot_tryton.org
1 month, 3 weeks ago (2018-07-27 07:33:43 UTC) #16
New changeset b72e1e9b9097 by Cédric Krier in branch 'default':
Use task queue to process sale
https://hg.tryton.org/modules/sale/rev/b72e1e9b9097
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766