Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Stefan Rijnhart (Therp)
Please stop merging so fast. If this is a submit of the current state of affairs on 7.0, there were all kinds of problems with it. See my review there. Serpent never addressed my comments directly but updated the branch periodically. I believe it has improved in certain aspects, but it was

[Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1 has been updated. Status: Merged = Work in progress For more details, see: https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619 --

Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Merge uncommited and MP reopened. -- https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619 Your team OpenERP Community is subscribed to branch lp:~openerp-community/server-env-tools/6.1-mass_editing. ___ Mailing

Re: [Openerp-community] city / base_city module for OpenERP v7, anyone?

2013-05-01 Thread Pedro Manuel Baeza Romero
Hi, Raphael: Here it is also merge proposal for 7.0 version: https://code.launchpad.net/~pedro.baeza/partner-contact-management/city-module/+merge/161837 I think it includes all features of better_zip module, so you don't need to use it. I'm going to propose to spanish localization to use it

Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Stefan Rijnhart (Therp)
Review: Needs Fixing My review is a bit compact but feel free to discuss anything that you disagree about. l. 128 e.a. Please replace the whole stringified ID list thing by a regular many2many field. Adapt onchange_model to return just a list of ids. Replace the eval'ed string clause in the

Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
@Jay Do you want me to fix it or you fix it yourself ? Thanks. -- https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619 Your team OpenERP Community is subscribed to branch lp:~openerp-community/server-env-tools/6.1-mass_editing.

Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Serpent Consulting Services
If I am not wrong, only 2 or 3 points remain to be fixed. You can take a look at our v7 merge proposal. You can use the latest code from that branch. I dont want you to waste time on fixing the thigs we alrready fixed. Rather I want you to suggest things we can improve, or you yourself can do

Re: [Openerp-community] [Merge] lp:~openerp-community/server-env-tools/6.1-mass_editing into lp:server-env-tools/6.1

2013-05-01 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Then we will wait for the merge in v7 to backport it to 6.1. -- https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619 Your team OpenERP Community is subscribed to branch lp:~openerp-community/server-env-tools/6.1-mass_editing.