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

Issue 1914045: Update product_price_list module for new Transaction (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 6 months ago by ced
Modified:
15 years, 6 months ago
Reviewers:
bch, pheller
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Fix indent #

Total comments: 1

Patch Set 3 : Rename ctx into context #

Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -56 lines) Patch
M INSTALL View 1 chunk +1 line, -1 line 0 comments Download
M price_list.py View 1 2 6 chunks +36 lines, -55 lines 0 comments Download

Messages

Total messages: 5
ced
15 years, 6 months ago (2010-08-05 10:16:56 UTC) #1
ced
15 years, 6 months ago (2010-08-05 10:18:24 UTC) #2
pheller
http://codereview.appspot.com/1914045/diff/3001/4002 File price_list.py (right): http://codereview.appspot.com/1914045/diff/3001/4002#newcode199 price_list.py:199: ctx = Transaction().context.copy() could be called "context" now
15 years, 6 months ago (2010-08-09 14:41:19 UTC) #3
ced
15 years, 6 months ago (2010-08-09 14:47:41 UTC) #4
pheller
15 years, 6 months ago (2010-08-09 14:55:15 UTC) #5
looks good here
Sign in to reply to this message.

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