openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~therp-nl/account-financial-tools/7.0-add_move_line_no_default_search_period_journal into lp:account-financial-tools

To: mp+207841@xxxxxxxxxxxxxxxxxx
From: Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>
Date: Mon, 03 Mar 2014 23:47:44 -0000
In-reply-to: <20140303151721.20329.87562.launchpad@ackee.canonical.com>
Reply-to: mp+207841@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review and test

Hi, Stefan,

I have tested the module and now I see no reason for the new menu I proposed, because it only changes default values. It doesn't remove combo-boxes, which is what I understand at the beginning.

It works perfectly.

Thanks for the contribution.

Regards.
-- 
https://code.launchpad.net/~therp-nl/account-financial-tools/7.0-add_move_line_no_default_search_period_journal/+merge/207841
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.

References

[Merge] lp:~therp-nl/account-financial-tools/7.0-add_move_line_no_default_search_period_journal into lp:account-financial-tools
From: Stefan Rijnhart (Therp), 2014-03-03