Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(30)

Issue 36921002: purchase: Link backorder moves with existing invoices lines on order shipment method (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 months ago by pokoli
Modified:
9 months ago
Reviewers:
rietveld-bot, ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : Do not test invoice state #

Unified diffs Side-by-side diffs Delta from patch set Stats (+44 lines, -1 line) Patch
M purchase.py View 1 1 chunk +4 lines, -1 line 0 comments Download
M tests/scenario_purchase.rst View 1 chunk +40 lines, -0 lines 0 comments Download

Messages

Total messages: 7
pokoli
11 months ago (2017-12-19 16:56:58 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/36921002
11 months ago (2017-12-19 17:13:25 UTC) #2
ced
https://codereview.tryton.org/36921002/diff/1/purchase.py File purchase.py (right): https://codereview.tryton.org/36921002/diff/1/purchase.py#newcode1482 purchase.py:1482: if l.invoice_state == 'draft'] The state of the invoice ...
11 months ago (2017-12-19 18:43:01 UTC) #3
pokoli
Do not test invoice state
11 months ago (2017-12-20 14:54:36 UTC) #4
reviewbot
flake8 OK URL: https://codereview.tryton.org/36921002
11 months ago (2017-12-20 15:11:58 UTC) #5
ced
LGTM
9 months, 1 week ago (2018-02-11 00:22:04 UTC) #6
rietveld-bot_tryton.org
9 months ago (2018-02-15 09:47:17 UTC) #7
New changeset 6281858203b2 by Sergi Almacellas Abellana in branch 'default':
Link backorder moves with existing invoices lines on order shipment method
http://hg.tryton.org/modules/purchase/rev/6281858203b2
Sign in to reply to this message.

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