Merge lp:~therp-nl/banking-addons/ba70-show_statement_line_date into lp:banking-addons

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Approved by: Holger Brunn (Therp)
Approved revision: 226
Merged at revision: 230
Proposed branch: lp:~therp-nl/banking-addons/ba70-show_statement_line_date
Merge into: lp:banking-addons
Diff against target: 12 lines (+1/-1)
1 file modified
account_banking/account_banking_view.xml (+1/-1)
To merge this branch: bzr merge lp:~therp-nl/banking-addons/ba70-show_statement_line_date
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) Approve
Raphaël Valyi - http://www.akretion.com Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+207470@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM, as extended group is no more available in v7.

Regards.

review: Approve (code review)
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

LGTM

review: Approve
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_banking/account_banking_view.xml'
2--- account_banking/account_banking_view.xml 2013-10-31 07:33:46 +0000
3+++ account_banking/account_banking_view.xml 2014-02-20 15:35:53 +0000
4@@ -321,7 +321,7 @@
5 <field name="arch" type="xml">
6 <tree string="Statement lines" colors="black:state == 'confirmed';darkmagenta:match_multi == True;crimson:duplicate == True;grey:state=='draft';">
7 <field name="sequence" readonly="1" invisible="1"/>
8- <field name="date" groups="base.group_extended"/>
9+ <field name="date"/>
10 <field name="name"/>
11 <field name="ref"/>
12 <field name="partner_id" on_change="onchange_partner_id(partner_id)"/>

Subscribers

People subscribed via source and target branches

to status/vote changes: