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

Issue 37571002: purchase: Ensure empty purchase goes to done state

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 5 days ago by ced
Modified:
1 week, 1 day ago
Reviewers:
pokoli, reviewbot
Visibility:
Public.

Description

As there is not transition from confirmed to done, the purchase must first be processed. issue6639

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+72 lines, -2 lines) Patch
M purchase.py View 2 chunks +5 lines, -2 lines 0 comments Download
M setup.py View 1 chunk +1 line, -0 lines 0 comments Download
A tests/scenario_purchase_empty.rst View 1 chunk +62 lines, -0 lines 0 comments Download
M tests/test_purchase.py View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 3
ced
1 week, 5 days ago (2017-07-13 09:08:21 UTC) #1
reviewbot
https://codereview.tryton.org/37571002/diff/1/setup.py#newcode88 setup.py:88: E501 line too long (85 > 79 characters) URL: https://codereview.tryton.org/37571002
1 week, 5 days ago (2017-07-13 09:33:23 UTC) #2
pokoli
1 week, 1 day ago (2017-07-17 09:59:42 UTC) #3
LGTM
Sign in to reply to this message.

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