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

Issue 97085: Make complete inventory lines overridable (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 8 months ago by ced1
Modified:
14 years, 8 months ago
Reviewers:
bch, bch, udono, yangoon1
Visibility:
Public.

Description

Improve extendibility of stock.inventory inspired by issue1093

Patch Set 1 #

Total comments: 3

Patch Set 2 : Fix typo and remove product_done #

Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -31 lines) Patch
M inventory.py View 1 2 chunks +61 lines, -31 lines 0 comments Download

Messages

Total messages: 6
ced1
14 years, 8 months ago (2009-07-23 17:59:35 UTC) #1
udono
Just a typo in docstring http://codereview.appspot.com/97085/diff/1/2 File inventory.py (right): http://codereview.appspot.com/97085/diff/1/2#newcode303 Line 303: # if nothing ...
14 years, 8 months ago (2009-07-23 18:20:29 UTC) #2
udono
last comment was wrong... http://codereview.appspot.com/97085/diff/1/2 File inventory.py (right): http://codereview.appspot.com/97085/diff/1/2#newcode303 Line 303: # if nothing as ...
14 years, 8 months ago (2009-07-23 18:23:27 UTC) #3
bch
http://codereview.appspot.com/97085/diff/1/2 File inventory.py (right): http://codereview.appspot.com/97085/diff/1/2#newcode171 Line 171: product_done = [] Why using product_done, there was ...
14 years, 8 months ago (2009-07-23 19:22:06 UTC) #4
ced1
14 years, 8 months ago (2009-07-23 20:28:43 UTC) #5
yangoon1
14 years, 8 months ago (2009-07-24 01:52:42 UTC) #6

          
Sign in to reply to this message.

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