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

Issue 70391002: account_statement: Split line also when amount to pay is zero

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 4 weeks ago by ced
Modified:
2 months ago
Reviewers:
reviewbot
Visibility:
Public.

Description

This is less astonishing for the user compare to remove the invoice he just selected. issue7707

Patch Set 1 #

Patch Set 2 : Remove unused code #

Patch Set 3 : Update to tip #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -24 lines) Patch
M CHANGELOG View 1 2 1 chunk +4 lines, -0 lines 0 comments Download
M statement.py View 1 2 7 chunks +17 lines, -24 lines 0 comments Download
M tests/scenario_account_statement.rst View 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 6
ced
2 months, 4 weeks ago (2018-09-18 17:23:48 UTC) #1
reviewbot
https://codereview.tryton.org/70391002/diff/1/statement.py#newcode12 statement.py:12: F401 'Check' imported but unused URL: https://codereview.tryton.org/70391002
2 months, 4 weeks ago (2018-09-18 17:29:08 UTC) #2
ced
Remove unused code
2 months, 4 weeks ago (2018-09-18 17:38:03 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/70391002
2 months, 4 weeks ago (2018-09-18 17:55:51 UTC) #4
ced
Update to tip
2 months ago (2018-10-17 13:06:26 UTC) #5
reviewbot
2 months ago (2018-10-17 13:06:50 UTC) #6
Sign in to reply to this message.

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