savoirfairelinux-openerp team mailing list archive

Thread
Date

Re: [Merge] lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

To: "Maxime Chambreuil \(http://www.savoirfairelinux.com\)" <maxime.chambreuil@xxxxxxxxxxxxxxxxxxxx>
From: Mario Arias <the.clone.master@xxxxxxxxx>
Date: Fri, 21 Mar 2014 10:10:20 -0000
In-reply-to: <20140107222716.3809.89445.launchpad@ackee.canonical.com>
Reply-to: mp+200744@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing code review and test

I just found two small issues:

import_odbc_view:
Line 45:  the "tree" is missing the "string" attribute, that is mandatory.  Just add 'string="Tree Name"' as attribute and the error will be gone.
Line 81:  base.next_id_15 is not present in OpenERP Saas3.  Not sure what menu was that, but this is fixed just defining what would be the right menu to place the module under...


Also it seems that base_external_dbsource branch in LP is more "up to date" than included module in this one...
I will check change history and diff, and will publish back any findings...

Regards,
Mario
-- 
https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744
Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension.