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

Issue 3371002: account_fr: Fix categorization of 4551 and 4558 accounts (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 6 months ago by RaphaelHertzog
Modified:
8 years, 5 months ago
Reviewers:
yangoon, ced
Visibility:
Public.

Description

account_fr: Fix categorization of 4551 and 4558 accounts

Patch Set 1 #

Patch Set 2 : account_fr: Fix categorization of 4551 and 4558 accounts #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M account_fr.xml View 1 chunk +4 lines, -4 lines 4 comments Download

Messages

Total messages: 9
RaphaelHertzog
8 years, 6 months ago (2014-02-10 10:17:47 UTC) #1
ced
Can not read the diff of the patch. It happens sometimes with rietveld, you must ...
8 years, 6 months ago (2014-02-10 11:07:41 UTC) #2
Paranoid Review Bot
flake8 OK
8 years, 6 months ago (2014-02-10 11:10:12 UTC) #3
RaphaelHertzog
8 years, 6 months ago (2014-02-10 11:10:24 UTC) #4
Paranoid Review Bot
flake8 OK
8 years, 6 months ago (2014-02-10 12:10:12 UTC) #5
yangoon
8 years, 6 months ago (2014-02-10 13:19:33 UTC) #6
ced
http://codereview.tryton.org/3371002/diff/20001/account_fr.xml File account_fr.xml (right): http://codereview.tryton.org/3371002/diff/20001/account_fr.xml#newcode5572 account_fr.xml:5572: <field name="type" ref="emprunts_dettes_financieres_diverses"/> But in OE, it is in ...
8 years, 6 months ago (2014-02-14 00:19:23 UTC) #7
RaphaelHertzog
http://codereview.tryton.org/3371002/diff/20001/account_fr.xml File account_fr.xml (right): http://codereview.tryton.org/3371002/diff/20001/account_fr.xml#newcode5572 account_fr.xml:5572: <field name="type" ref="emprunts_dettes_financieres_diverses"/> On 2014/02/14 00:19:24, ced wrote: > ...
8 years, 6 months ago (2014-02-14 07:03:42 UTC) #8
ced
8 years, 6 months ago (2014-02-14 09:52:31 UTC) #9
LGTM
Sign in to reply to this message.

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