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

Issue 46471002: analytic_invoice: Add company domain on asset analytic accounts

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 months, 1 week ago by pokoli
Modified:
10 months ago
Reviewers:
ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : Add company domain on asset analytic lines #

Total comments: 1

Patch Set 3 : Add default integer value on company eval #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M asset.py View 1 2 2 chunks +9 lines, -0 lines 1 comment Download

Messages

Total messages: 9
pokoli
10 months, 1 week ago (2018-05-21 10:02:12 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/46471002
10 months, 1 week ago (2018-05-21 10:20:01 UTC) #2
ced
https://codereview.tryton.org/46471002/diff/1/invoice.py File invoice.py (right): https://codereview.tryton.org/46471002/diff/1/invoice.py#newcode17 invoice.py:17: analytic_company = fields.Function(fields.Many2One( In account_invoice, there is already a ...
10 months ago (2018-05-23 21:44:01 UTC) #3
pokoli
Add company domain on asset analytic lines
10 months ago (2018-05-24 08:26:31 UTC) #4
ced
https://codereview.tryton.org/46471002/diff/20001/asset.py File asset.py (right): https://codereview.tryton.org/46471002/diff/20001/asset.py#newcode20 asset.py:20: ('company', '=', Eval('company')), Eval should have a integer as ...
10 months ago (2018-05-24 08:32:32 UTC) #5
reviewbot
flake8 OK URL: https://codereview.tryton.org/46471002
10 months ago (2018-05-24 08:49:24 UTC) #6
pokoli
Add default integer value on company eval
10 months ago (2018-05-24 08:50:58 UTC) #7
ced
https://codereview.tryton.org/46471002/diff/40001/asset.py File asset.py (right): https://codereview.tryton.org/46471002/diff/40001/asset.py#newcode22 asset.py:22: cls.analytic_accounts.depends.append('company') Should not we have a on_change_company to remove ...
10 months ago (2018-05-24 09:14:33 UTC) #8
reviewbot
10 months ago (2018-05-24 09:21:42 UTC) #9
Sign in to reply to this message.

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