Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(12159)
Issues Repositories Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

Issue 4214045: New module sale_shipment_cost (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by ced
Modified:
12 years, 8 months ago
Reviewers:
bch, albertnan, yangoon, nicoe
Visibility:
Public.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Update to 2.0 and add Shipment Cost Method #

Patch Set 3 : Remove shipping naming, set always add the shipment cost at the bottom #

Patch Set 4 : Fix scenario #

Patch Set 5 : Add on_change on inventory_moves #

Patch Set 6 : Add test on stockable product, add doc #

Total comments: 2

Patch Set 7 : Fix typo #

Patch Set 8 : Fix editing cost line #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1706 lines, -0 lines) Patch
A COPYRIGHT View 1 1 chunk +17 lines, -0 lines 0 comments Download
A INSTALL View 1 1 chunk +33 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 4 5 6 1 chunk +36 lines, -0 lines 0 comments Download
A __init__.py View 1 1 chunk +5 lines, -0 lines 0 comments Download
A __tryton__.py View 1 1 chunk +25 lines, -0 lines 0 comments Download
A doc/index.rst View 1 2 3 4 5 1 chunk +20 lines, -0 lines 0 comments Download
A sale.py View 1 2 3 4 5 6 7 1 chunk +242 lines, -0 lines 0 comments Download
A sale.xml View 1 2 3 4 5 1 chunk +59 lines, -0 lines 0 comments Download
A setup.py View 1 1 chunk +66 lines, -0 lines 0 comments Download
A stock.py View 1 2 3 4 1 chunk +155 lines, -0 lines 0 comments Download
A stock.xml View 1 2 1 chunk +28 lines, -0 lines 0 comments Download
A tests/__init__.py View 1 1 chunk +4 lines, -0 lines 0 comments Download
A tests/scenario_sale_shipment_cost.rst View 1 2 3 4 5 6 7 1 chunk +279 lines, -0 lines 0 comments Download
A tests/test_sale_shipment_cost.py View 1 4 1 chunk +53 lines, -0 lines 0 comments Download

Messages

Total messages: 12
ced
13 years, 1 month ago (2011-02-24 08:10:21 UTC) #1
yangoon
http://codereview.appspot.com/4214045/diff/1/__tryton__.py File __tryton__.py (right): http://codereview.appspot.com/4214045/diff/1/__tryton__.py#newcode17 __tryton__.py:17: 'account_invoice', Question: Do we have a guidline how to ...
12 years, 10 months ago (2011-05-31 10:52:37 UTC) #2
ced
http://codereview.appspot.com/4214045/diff/1/__tryton__.py File __tryton__.py (right): http://codereview.appspot.com/4214045/diff/1/__tryton__.py#newcode17 __tryton__.py:17: 'account_invoice', On 2011/05/31 10:52:37, yangoon wrote: > Question: Do ...
12 years, 10 months ago (2011-05-31 10:58:39 UTC) #3
ced
12 years, 10 months ago (2011-06-22 14:55:24 UTC) #4
ced
12 years, 9 months ago (2011-07-02 12:46:16 UTC) #5
ced
12 years, 9 months ago (2011-07-09 22:35:52 UTC) #6
ced
12 years, 9 months ago (2011-07-09 23:47:42 UTC) #7
ced
12 years, 9 months ago (2011-07-10 22:06:44 UTC) #8
albertnan
http://codereview.appspot.com/4214045/diff/39002/README File README (right): http://codereview.appspot.com/4214045/diff/39002/README#newcode4 README:4: The party module of the Tryton application platform. Should ...
12 years, 9 months ago (2011-07-11 00:00:30 UTC) #9
ced
12 years, 9 months ago (2011-07-11 07:29:08 UTC) #10
ced
http://codereview.appspot.com/4214045/diff/39002/README File README (right): http://codereview.appspot.com/4214045/diff/39002/README#newcode4 README:4: The party module of the Tryton application platform. On ...
12 years, 9 months ago (2011-07-11 07:29:17 UTC) #11
ced
12 years, 9 months ago (2011-07-24 16:09:27 UTC) #12 

          

          

            
              Sign in to reply to this message.
            
          

        


      

    

    

  













      		
    

  





  
    
  






  Powered by Google App Engine





  

  RSS Feeds
  Recent Issues
  
  
  |
  This issue
  
  

  
This is Rietveld f62528b