Merge lp:~pablocm/account-payment/7.0-account-payment-extension-groups-fix into lp:~account-payment-team/account-payment/7.0

Proposed by PabloCM
Status: Merged
Merged at revision: 110
Proposed branch: lp:~pablocm/account-payment/7.0-account-payment-extension-groups-fix
Merge into: lp:~account-payment-team/account-payment/7.0
Diff against target: 15 lines (+2/-2)
1 file modified
account_payment_extension/account_payment_view.xml (+2/-2)
To merge this branch: bzr merge lp:~pablocm/account-payment/7.0-account-payment-extension-groups-fix
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp code review Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+203677@code.launchpad.net

Description of the change

Removed the groups attribute on the field reconcile_partial_id and changed it to multi currency on currency_id on the account.move.line tree view. They were referencing the no longer existant base.group_extended.

To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM

review: Approve (code review)
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) :
review: Approve (code review)
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I'll proceed to do the merge due to the two approvals and to have passed enough time from the proposal.

Regards.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account_payment_extension/account_payment_view.xml'
--- account_payment_extension/account_payment_view.xml 2013-08-18 21:18:02 +0000
+++ account_payment_extension/account_payment_view.xml 2014-01-29 08:38:15 +0000
@@ -29,9 +29,9 @@
29 <field name="amount_to_pay"/>29 <field name="amount_to_pay"/>
30 </field>30 </field>
31 <field name="currency_id" position="replace">31 <field name="currency_id" position="replace">
32 <field name="currency_id" groups="base.group_extended"/>32 <field name="currency_id" groups="base.group_multi_currency"/>
33 <field name="reconcile_id"/>33 <field name="reconcile_id"/>
34 <field name="reconcile_partial_id" groups="base.group_extended"/>34 <field name="reconcile_partial_id"/>
35 <field name="state"/>35 <field name="state"/>
36 <field name="payment_type"/>36 <field name="payment_type"/>
37 </field>37 </field>

Subscribers

People subscribed via source and target branches