Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(12)

Issue 4274055: Set shipment/invoice method as configuration option (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years ago by nicoe
Modified:
15 years ago
Reviewers:
yangoon, ced
Visibility:
Public.

Patch Set 1 #

Total comments: 5

Patch Set 2 : removed unused import #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -3 lines) Patch
M configuration.py View 1 2 chunks +15 lines, -1 line 0 comments Download
M configuration.xml View 2 chunks +17 lines, -0 lines 0 comments Download
M sale.py View 1 chunk +6 lines, -2 lines 0 comments Download

Messages

Total messages: 7
nicoe
15 years ago (2011-03-15 15:56:41 UTC) #1
ced
http://codereview.appspot.com/4274055/diff/1/configuration.py File configuration.py (right): http://codereview.appspot.com/4274055/diff/1/configuration.py#newcode4 configuration.py:4: from trytond.pyson import Eval, Not, Equal, Bool Not is ...
15 years ago (2011-03-15 16:14:03 UTC) #2
yangoon
http://codereview.appspot.com/4274055/diff/1/configuration.py File configuration.py (right): http://codereview.appspot.com/4274055/diff/1/configuration.py#newcode30 configuration.py:30: })) Like on purchase: Often used fields, that could ...
15 years ago (2011-03-15 22:09:11 UTC) #3
ced
http://codereview.appspot.com/4274055/diff/1/configuration.py File configuration.py (right): http://codereview.appspot.com/4274055/diff/1/configuration.py#newcode30 configuration.py:30: })) On 2011/03/15 22:09:11, yangoon wrote: > Like on ...
15 years ago (2011-03-15 22:15:22 UTC) #4
nicoe
http://codereview.appspot.com/4274055/diff/1/configuration.py File configuration.py (right): http://codereview.appspot.com/4274055/diff/1/configuration.py#newcode4 configuration.py:4: from trytond.pyson import Eval, Not, Equal, Bool On 2011/03/15 ...
15 years ago (2011-03-16 11:18:39 UTC) #5
nicoe
15 years ago (2011-03-16 11:19:19 UTC) #6
ced
15 years ago (2011-03-25 10:10:04 UTC) #7
LGTM
Sign in to reply to this message.

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