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

Issue 4217043: New module carrier (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 9 months ago by ced
Modified:
9 years, 4 months ago
Reviewers:
bch, albertnan, yangoon, nicoe
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Update to 2.0 and split cost computation into purchase and sale #

Patch Set 3 : Rename carrier_cost into carrier_cost_method, add doc #

Total comments: 2

Patch Set 4 : Fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1112 lines, -0 lines) Patch
A COPYRIGHT View 1 1 chunk +17 lines, -0 lines 0 comments Download
A INSTALL View 1 1 chunk +31 lines, -0 lines 0 comments Download
A LICENSE View 1 1 chunk +674 lines, -0 lines 0 comments Download
A MANIFEST.in View 1 1 chunk +10 lines, -0 lines 0 comments Download
A README View 1 2 3 1 chunk +36 lines, -0 lines 0 comments Download
A __init__.py View 1 1 chunk +4 lines, -0 lines 0 comments Download
A __tryton__.py View 1 1 chunk +22 lines, -0 lines 0 comments Download
A carrier.py View 1 2 1 chunk +62 lines, -0 lines 0 comments Download
A carrier.xml View 1 2 1 chunk +134 lines, -0 lines 0 comments Download
A doc/index.rst View 1 2 1 chunk +15 lines, -0 lines 0 comments Download
A setup.py View 1 1 chunk +66 lines, -0 lines 0 comments Download
A tests/__init__.py View 1 1 chunk +4 lines, -0 lines 0 comments Download
A tests/test_carrier.py View 1 1 chunk +37 lines, -0 lines 0 comments Download

Messages

Total messages: 7
ced
9 years, 9 months ago (2011-02-24 08:09:23 UTC) #1
yangoon
9 years, 6 months ago (2011-05-31 10:52:48 UTC) #2
ced
9 years, 5 months ago (2011-06-22 14:54:42 UTC) #3
ced
9 years, 4 months ago (2011-07-10 14:22:32 UTC) #4
albertnan
http://codereview.appspot.com/4217043/diff/14001/README File README (right): http://codereview.appspot.com/4217043/diff/14001/README#newcode4 README:4: The party module of the Tryton application platform. Should ...
9 years, 4 months ago (2011-07-11 00:00:08 UTC) #5
ced
9 years, 4 months ago (2011-07-11 07:28:13 UTC) #6
ced
9 years, 4 months ago (2011-07-11 07:28:21 UTC) #7
http://codereview.appspot.com/4217043/diff/14001/README
File README (right):

http://codereview.appspot.com/4217043/diff/14001/README#newcode4
README:4: The party module of the Tryton application platform.
On 2011/07/11 00:00:08, albertnan wrote:
> Should be 'The trytond_carrier module'

Done.
Sign in to reply to this message.

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