openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~serpentcs/server-env-tools/base_module_record into lp:server-env-tools

To: mp+196613@xxxxxxxxxxxxxxxxxx
From: Alexandre Fayolle - camptocamp <alexandre.fayolle@xxxxxxxxxxxxxx>
Date: Mon, 23 Jun 2014 09:43:56 -0000
In-reply-to: <20131125184224.518.52089.launchpad@ackee.canonical.com>
Reply-to: mp+196613@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review, no tests

Never used it, code looks shaggy, and there are not tests. 

Are there known issues? 

In it's current state, this module is not on par with the standards of OCA and it needs care and love. 

If someone has use for this module and is volunteering to maintain it, I'm willing to perform a detailed review of the code and help that person to improve the module. 
-- 
https://code.launchpad.net/~serpentcs/server-env-tools/base_module_record/+merge/196613
Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools.

References

[Merge] lp:~serpentcs/server-env-tools/base_module_record into lp:server-env-tools
From: Serpent Consulting Services, 2013-11-25