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

Issue 50431002: account_statement: Remove company header/footer from statement report

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

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -106 lines) Patch
M __init__.py View 1 chunk +0 lines, -3 lines 0 comments Download
M statement.fodt View 8 chunks +75 lines, -96 lines 0 comments Download
M statement.py View 2 chunks +1 line, -7 lines 1 comment Download

Messages

Total messages: 3
pokoli
4 years, 8 months ago (2018-05-24 10:41:43 UTC) #1
reviewbot
https://codereview.tryton.org/50431002/diff/1/__init__.py#newcode5 __init__.py:5: F403 'from journal import *' used; unable to detect undefined names https://codereview.tryton.org/50431002/diff/1/__init__.py#newcode6 __init__.py:6: ...
4 years, 8 months ago (2018-05-24 10:54:45 UTC) #2
ced
4 years, 8 months ago (2018-05-31 15:58:10 UTC) #3
https://codereview.tryton.org/50431002/diff/1/statement.py
File statement.py (left):

https://codereview.tryton.org/50431002/diff/1/statement.py#oldcode1100
statement.py:1100: __name__ = 'account.statement'
It is still good to have a report class, this allows extension.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld d9ca037-tainted