← Back to team overview

openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~pedro.baeza/account-financial-tools/7.0-account_chart_update-enhanced into lp:account-financial-tools

To: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
From: "Yannick Vaucher @ Camptocamp" <yannick.vaucher@xxxxxxxxxxxxxx>
Date: Tue, 20 May 2014 08:08:28 -0000
In-reply-to: <20140321011421.26179.58147.launchpad@ackee.canonical.com>
Reply-to: mp+212074@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review, no test

Seems fine that way ondelete="cascade" seems also a better choice.

Thanks!

LGTM
-- 
https://code.launchpad.net/~pedro.baeza/account-financial-tools/7.0-account_chart_update-enhanced/+merge/212074
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.

References

[Merge] lp:~pedro.baeza/account-financial-tools/7.0-account_chart_update-enhanced into lp:account-financial-tools
From: Pedro Manuel Baeza, 2014-03-21