← Back to team overview

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: "Guewen Baconnier @ Camptocamp" <guewen.baconnier@xxxxxxxxxxxxxx>
Date: Fri, 21 Mar 2014 15:38:43 -0000
In-reply-to: <20140303151721.20329.87562.launchpad@ackee.canonical.com>
Reply-to: mp+207841@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve code review

We thought about doing the same module - we are annoyed by this default behavior - and you did it! So nice ;-)

LGTM

Thanks
-- 
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