Re: [Openerp-connector-community] adding product to magento via OpenERP

2014-05-16 Thread Adam Langowski / IDEA07 Studio Kreatywne
Hello, can you explain how catalog should export product to magento? It is cron job, button or connector job or what? I'm testing it right now, but I haven't seen any option to do it and, in result, product stays only in openERP. Thanks in advance Kind Regards Adam Langowski W dniu

Re: [Openerp-connector-community] adding product to magento via OpenERP

2014-05-16 Thread David Beal
product must have a binding to the magento backend = connector tab David BEAL Akretion OpenERP Development - Integration +33 (0)6 67 22 86 89 +33 (0)4 82 53 84 60 2014-05-16 11:41 GMT+02:00 Adam Langowski / IDEA07 Studio Kreatywne adam.langow...@idea07.pl: Hello, can you explain how

Re: [Openerp-connector-community] adding product to magento via OpenERP

2014-05-16 Thread David Beal
after binding the product to backend, for each create/write event a job is created David BEAL Akretion OpenERP Development - Integration +33 (0)6 67 22 86 89 +33 (0)4 82 53 84 60 2014-05-16 11:44 GMT+02:00 David Beal david.b...@akretion.com: product must have a binding to the magento backend

Re: [Openerp-connector-community] adding product to magento via OpenERP

2014-05-16 Thread Landis Arnold
Davide, Question from the side here: Doing so requires the following correct, or is this in the standard connector now? Inventory is updating for me but I don't think I am updating descriptions now. Certainly not images yet. Magento changes to go through generally to OpenERP (- more side --

Re: [Openerp-community] Odoo and community

2014-05-16 Thread David Beal
Dear Odoo brand keeper, Could you provide an .svg format, the standard portable vector format. eps != linux Thanks David BEAL Akretion OpenERP Development - Integration +33 (0)6 67 22 86 89 +33 (0)4 82 53 84 60 2014-05-15 22:57 GMT+02:00 Carlos Vásquez carlos.vasq...@clearcorp.co.cr: Hi

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Ana Juaristi
+1 to proposal. We use inkscape 2014-05-16 9:29 GMT+02:00 David Beal david.b...@akretion.com: Dear Odoo brand keeper, Could you provide an .svg format, the standard portable vector format. eps != linux Thanks David BEAL Akretion OpenERP Development - Integration +33 (0)6 67 22 86 89

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Parthiv Patel
Shouldn't there be a standard guidelines to use logo, color shades and proportions as well ? Some sort of media kit like this : http://www.prestashop.com/en/media-kit On Fri, May 16, 2014 at 2:08 PM, Pedro Manuel Baeza Romero pedro.ba...@gmail.com wrote: Here it is a zip with SVG files that

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Alejandro Santana
Hi. Nice to hear about new updated documentation. I mean: It is a must! Good documentation: - Saves a lot developer time searching info in weird places or by trial-error. - Saves Odoo time fixing, reviewing bugs. - Makes the community stronger, more productive and happier. Thus,

[Openerp-community] Configuration issue and very silly blocking point

2014-05-16 Thread Ana Juaristi
Hello Everybody: We are trying to configure several domains related to spanish-communty against their own database with a single Odoo instance on machine. We are blocked. The issue is something/somewhere is marking by default to look for the database www (or what is before the domail name) on

Re: [Openerp-community] Configuration issue and very silly blocking point

2014-05-16 Thread Ana Juaristi
OK... autoanswer. Newbie thing We were testing on localhost and we had the issue. But in real server it seems it works right. Close the ticket ;P Thank you: Ana 2014-05-16 12:36 GMT+02:00 Ana Juaristi ajuaris...@gmail.com: Hello Everybody: We are trying to configure several domains

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Stefan Rijnhart
On 16-05-14 13:43, Alejandro Santana wrote: Also: Arista font is here: http://www.dafont.com/es/arista.font FYI: the license indicates that the font must be bought for commercial purposes. Regards, Stefn. -- Therp - Maatwerk in open ontwikkeling Stefan Rijnhart - Ontwerp en

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Niels Huylebroeck
There is such a page : https://www.odoo.com/page/brand-assets 2014-05-16 11:31 GMT+02:00 Parthiv Patel parthiv...@gmail.com: Shouldn't there be a standard guidelines to use logo, color shades and proportions as well ? Some sort of media kit like this : http://www.prestashop.com/en/media-kit

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Niels Huylebroeck
Time for a first investment of odoo's new $10m: http://www.zetafonts.com/collection/8/purchasing_options 2014-05-16 14:28 GMT+02:00 Stefan Rijnhart ste...@therp.nl: On 16-05-14 13:43, Alejandro Santana wrote: Also: Arista font is here: http://www.dafont.com/es/arista.font FYI: the

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Robin
I missed the discussion relating to this name change. Somebody has been smoking some strong weed. This is one of the daftess names I have ever come across. In English, it reminds readers of a *BAD SMELL* - ODOO - ODOUR - ODOR , you could say that ODOO is only 1 R(se) away from a bad ODOR

Re: [Openerp-community] Odoo and community

2014-05-16 Thread AKSANA
Odoo reminds to me Hodor from Games of Thrones, the retarded guy... Em 16-05-2014 14:47, Robin escreveu: I missed the discussion relating to this name change. Somebody has been smoking some strong weed. This is one of the daftess names I have ever come across. In English, it reminds readers

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Enapps :: Vadim Chobanu
Join the club Robin! its how openerp sa implements these sort of strategic choices and decisions. Just do them recklessly. Cheers, Vadim Sent from my Windows Phone -- From: Robin robin...@live.co.uk Sent: ‎16/‎05/‎2014 14:59 To:

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Marcelo Bello
But but but... the more O's in the name, the higher the valuation! And you know, with so many VCs on board, valuation is all that matters. (I am right thinking that the number of O's argument was just a silly joke and is not really the basis for the naming, right?) Source:

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Marcelo Bello
Actually, I think I will fork it and call it yabadabado. With so many O's in it, I can already feel the money coming. On Fri, May 16, 2014 at 11:12 AM, Enapps :: Vadim Chobanu va...@enapps.co.uk wrote: Join the club Robin! its how openerp sa implements these sort of strategic

Re: [Openerp-community] Odoo and community

2014-05-16 Thread abhishek jaiswal
Hi all, I am new to odoo, openerp and openesource too and i wanted to be a community member for odoo accounting and sales module what steps should i foloow to become one Thanks all in advance On Fri, May 16, 2014 at 8:04 PM, Marcelo Bello marcelo.be...@gmail.comwrote: Actually, I think I

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Enapps :: Vadim Chobanu
Weirdly enough, ‘Jack of all trades and master of none’ also 3 O’s... *From:* Marcelo Bello [mailto:marcelo.be...@gmail.com] *Sent:* 16 May 2014 15:34 *To:* Enapps :: Vadim Chobanu *Cc:* Robin; openerp-community@lists.launchpad.net *Subject:* Re: [Openerp-community] Odoo and community

[Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Joël Grand-Guillaume
Dear community, After the announcement of the brand new odoo instead of OpenERP, the main changes for contributors currently is the switch to Github. As I know, most of you waited for such a move ! Now it's done :) Though it remain a quite huge question : How the hell do we, as the community,

[Openerp-community] Playing games with names

2014-05-16 Thread W. Martin Borgert
Quoting Enapps :: Vadim Chobanu va...@enapps.co.uk: Weirdly enough, ‘Jack of all trades and master of none’ also 3 O’s... $ echo 'Jack of all trades, master of none' | sed 's/[^do]//g' odoo $ echo 'Llanfairpwllgwyngyllgogerychwyrndrobwantysiliogogogoch' | sed 's/[^do]//g' odo So

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Stefan Rijnhart
Hi Joël, Maybe I don't understand, but having one addon per repo does that mean that when an API change requires a change in all the banking addons (for instance), that I have to make a pull request for all those repos? I'd be in favour of keeping OCA projects in bzr for 7.0 and in GitHub for

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Leonardo Pistone
I vote for a) as the quickest and easiest option to arrive to the full split for v8. (unless someone can manage to do the split of v7 in a few days, but I suppose that's not possible). On Fri, May 16, 2014 at 5:31 PM, Ovnicraft ovnicr...@gmail.com wrote: Hi Joel ! On Fri, May 16, 2014 at

Re: [Openerp-community] [Openerp-community-reviewer] OCA LP Project migration to Github

2014-05-16 Thread Pedro Manuel Baeza Romero
I think Stefan raises an interesting approach: let v7 as is on Launchpad, and move quietly to GitHub for v8 modules (one module migration, a module that is tested on v8 and works, etc) in the form 1 module/1 repo. About the other problem, I don't know exactly GitHub, but I think you can propose a

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Ovnicraft
Hi Joel ! On Fri, May 16, 2014 at 10:16 AM, Joël Grand-Guillaume joel.grandguilla...@camptocamp.com wrote: Dear community, After the announcement of the brand new odoo instead of OpenERP, the main changes for contributors currently is the switch to Github. As I know, most of you waited

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Sebastien Beau
vote : A Big thanks to Joel for this mail ;) 2014-05-16 17:16 GMT+02:00 Joël Grand-Guillaume joel.grandguilla...@camptocamp.com: Dear community, After the announcement of the brand new odoo instead of OpenERP, the main changes for contributors currently is the switch to Github. As I

Re: [Openerp-community] [Openerp-community-reviewer] OCA LP Project migration to Github

2014-05-16 Thread Sylvain LE GAL
Hi all, @Joel : thanks a lot for your mail. The d) solution (of Stefan) is maybe the most interesting. There is currently no V8 serie for community modules in Launchpad (or not a lot ?) so realize the migration for the V8 only will avoid to move manually MP from LP to Github, to loose reviews or

Re: [Openerp-community] Odoo and community

2014-05-16 Thread jeff.wang
also this fantastic new name sounds silly in CHINESE, no one know how to spell it then we found this http://leloimg.lelo.netdna-cdn.com/graphic/LELO-Soraya-deep-rose-dual-action-vibrator.jpg -- Jeff Wang | j...@osbzr.com | 18016291663 | 02158980787 @OpenERP_Jeff As

Re: [Openerp-community] [Openerp-community-reviewer] OCA LP Project migration to Github

2014-05-16 Thread Ana Juaristi
+1 to option D: let = V7 on launchpad and only create needed V8 projects on Github. Less work to manage: new version, new everything 2014-05-16 18:12 GMT+02:00 Sylvain LE GAL sylvain.le...@grap.coop: Hi all, @Joel : thanks a lot for your mail. The d) solution (of Stefan) is maybe the most

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Martin Collins
On 2014-05-16 09:36, jeff.wang wrote: also this fantastic new name sounds silly in CHINESE, no one know how to spell it then we found this http://leloimg.lelo.netdna-cdn.com/graphic/LELO-Soraya-deep-rose-dual-action-vibrator.jpg That's rather beautiful. I wonder if it will interface with

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Bidoul , Stéphane
Stefan's approach has my vote. Keep = 7.0 on lauchpad and use github for = 8.0. Cheers, -sbi On Fri, May 16, 2014 at 5:51 PM, Stefan Rijnhart ste...@therp.nl wrote: Hi Joël, Maybe I don't understand, but having one addon per repo does that mean that when an API change requires a change

[Openerp-community] How will price per user really work?

2014-05-16 Thread Marcelo Bello
So let's say I have odoo with 8 app bundles (ecommerce, accounting, sales, wms, etc) and some 20 users in total. Then a total of 4 workers (users) at the warehouse, using only the wms app bundle. For those 4 users do I pay for only what they actually use (wms app bundle) or do I pay for all 8

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Ana Juaristi
Really very good question!!! if you make quotation for your configuration: 8 * 20 * 12 = 1920€/month If pricing was really user/app by moth would be more affordable and easier to sale. I think it should be commented to OpenERP, SA next community days Another real situation could be for instance:

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Maxime Chambreuil
We agree with Stefan here. -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Stefan's approach has my vote. Keep = 7.0 on lauchpad and use github for = 8.0. Cheers, -sbi On Fri, May 16, 2014 at 5:51 PM, Stefan Rijnhart ste...@therp.nl wrote: Hi Joël,

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Marcelo Bello
Hummm I just watched a video that was presented to partners in the US and they say we will pay for all apps installed and not apps used. In my eyes it doesn't make any sense. Also, this thing that partners will not get any commission on contracts with more than two years is REALLY nonsense.

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Gianluca Milano
Sadly, as written here https://www.odoo.com/page/FAQ-pricing it’s 20*8*12 €/month... Gianluca Milano | Business Integrator +39 081 194 65 159 | +39 335 60 10 512 gianlucamilano | Via Trepiccioni, 39 - 80078 Pozzuoli (NA) - ITALY P.IVA IT07227561219 Il giorno

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Lorenzo Battistini
On 05/16/2014 06:13 PM, Marcelo Bello wrote: So let's say I have odoo with 8 app bundles (ecommerce, accounting, sales, wms, etc) and some 20 users in total. Then a total of 4 workers (users) at the warehouse, using only the wms app bundle. For those 4 users do I pay for only what they

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Ovnicraft
On Fri, May 16, 2014 at 10:51 AM, Stefan Rijnhart ste...@therp.nl wrote: Hi Joël, Maybe I don't understand, but having one addon per repo does that mean that when an API change requires a change in all the banking addons (for instance), that I have to make a pull request for all those

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Marcelo Bello
I think it is not a question of whether or not it is more expensive. With all the new funding you can bet it will all become more expensive. The question is whether or not you can convince your client that this pricing scheme makes any sense. Pricing is Rally simple won't work as an argument

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Enapps :: Vadim Chobanu
You meant to say that the new pricing is more expensive than SAP cloud offering? -Original Message- From: Openerp-community [mailto:openerp-community-bounces+vadim=enapps.co...@lists.launchpad.net] On Behalf Of Lorenzo Battistini Sent: 16 May 2014 17:56 To:

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Gianluca Milano
…and the 2 years commission is the icing on the cake! Gianluca Milano | Business Integrator +39 081 194 65 159 | +39 335 60 10 512 gianlucamilano | Via Trepiccioni, 39 - 80078 Pozzuoli (NA) - ITALY P.IVA IT07227561219 Il giorno 16/mag/2014, alle ore 18:47,

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Davide Corio
On 16 May 2014, at 18:55, Lorenzo Battistini lorenzo.battist...@agilebg.com wrote: It would make much more sense if you paid for only what the users use. +1 Otherwise it means that odoo is more expensive than openerp, and less saleable. It is. As per our customers average size and

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Daniel Reis
Make your calculations here: https://accounts.openerp.com/page/pricing /DR ___ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community More help

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Dominique Chabord
If the product is as marvellous as is said, then x3 on unit price should result in x3 on partners business, and customers who spent two years I read somewhere : we had a deep thought about prices and we are ahead a very stable period ;-) If you get three users : According to Xavier, the sales

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Raphael Valyi
On Fri, May 16, 2014 at 1:47 PM, Marcelo Bello marcelo.be...@gmail.comwrote: Hummm I just watched a video that was presented to partners in the US and they say we will pay for all apps installed and not apps used. In my eyes it doesn't make any sense. Also, this thing that partners will

Re: [Openerp-community] How will price per user really work?

2014-05-16 Thread Marcelo Bello
And they are of course ignoring that not all markets can afford the same. Me, I have 20 users and there is no way I will pay EUR 1800/month plus 40% tax if I really need to contract with Odoo overseas (service imports are taxed heavily in Brazil). And then it all becomes much more expensive

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Eva Pinter
The name is definitively badly chosen, In Ireland, DooDoo is used by small kids to say excrement. We had a presentation today and the people were quite shocked over the new name of the software. (rofl) On 16 May 2014, at 15:23, AKSANA aksana.ivan...@exeq.pt wrote: Odoo reminds to me Hodor

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Raphael Valyi
On Fri, May 16, 2014 at 3:56 PM, Eva Pinter epin...@openit.ch wrote: The name is definitively badly chosen, In Ireland, DooDoo is used by small kids to say excrement. We had a presentation today and the people were quite shocked over the new name of the software. (rofl) Hard to please

Re: [Openerp-community] OCA LP Project migration to Github

2014-05-16 Thread Eric Caudal
d) or c) at least as first move (it could be done in 2 phases) or unless apps.odoo.com does not support lp anymore. Eric CAUDAL Eric Caudal /CEO/ -- *Elico Corporation, Shanghai branch /OpenERP Premium Certified Training Partner/ * Cell: + 86 186 2136 1670 Office: + 86 21 6211 8017/27/37

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Yury Tello
In the morning a friend confuses Odoo with odio (hate in spanish) :) 2014-05-16 14:07 GMT-05:00 Raphael Valyi rva...@gmail.com: On Fri, May 16, 2014 at 3:56 PM, Eva Pinter epin...@openit.ch wrote: The name is definitively badly chosen, In Ireland, DooDoo is used by small kids to say

[Openerp-community] Transition to GitHub

2014-05-16 Thread Olivier Dony
Hi everyone, The transition from Launchpad to GitHub has just been announced, and it raises many questions. First of all: Don't Panic!(TM) We should not lose anything in the process, and no urgent action is required on your side. The new Odoo wiki at GitHub attempts to give a comprehensive

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Open Source
The naming research came from a top ten list on Wikipedia? On 17 May 2014 00:28, Marcelo Bello marcelo.be...@gmail.com wrote: But but but... the more O's in the name, the higher the valuation! And you know, with so many VCs on board, valuation is all that matters. (I am right thinking that

Re: [Openerp-community] Transition to GitHub

2014-05-16 Thread Carlos Vásquez
Very good job with the wiki. Thanks. It's nice to see you working after hours to pave the way. Olivier, can you share with us what would be your advice for our branches? One repo per app bundle, one repo per module? Also, what is your recommendation for odoo localization repos naming? And for

Re: [Openerp-community] Odoo and community

2014-05-16 Thread Sudheendra Walvekar
In one of the Indian languages in the location where I live, Bangalore, known for largest Industries in IT sector in the world, the meaning of odoo is run away. :). Wondering, what I am supposed to do after associating with openERP for over 2 years. Sudheendra Walvekar Prime Minds

[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls into lp:report-print-send

2014-05-16 Thread Lionel Sausin - Numérigraphe
Lionel Sausin - Numérigraphe has proposed merging lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls into lp:report-print-send. Requested reviews: Report Printing and Sending Core Editors (report-print-send-core-editors) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls into lp:report-print-send

2014-05-16 Thread Pedro Manuel Baeza
Review: Approve code review LGTM. Regards. -- https://code.launchpad.net/~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls/+merge/219787 Your team Report Printing and Sending Core Editors is subscribed to branch lp:report-print-send. -- Mailing list:

[Openerp-community-reviewer] [Bug 1256199] Re: [7.0] Ubuntu 13.10 Conflict with python package resource

2014-05-16 Thread Lionel Sausin - Numérigraphe
The branches attached seem to work, is it worth merging them in OCB ? ** Also affects: ocb-addons Importance: Undecided Status: New ** Also affects: ocb-server Importance: Undecided Status: New ** Also affects: ocb-server/7.0 Importance: Undecided Status: New **

[Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1

2014-05-16 Thread Yann Papouin
Yann Papouin has proposed merging lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1. Requested reviews: OpenERP Community Backports (ocb) Related bugs: Bug #1008513 in OpenERP Community Backports (Addons): [6.1]document_webdav - pywebdav import error if

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1

2014-05-16 Thread Pedro Manuel Baeza
Review: Needs Fixing code review I think you may keep osv.except_osv in case both imports fail, or put an external dependency on manifest. Regards. -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error/+merge/219794 Your team OpenERP Community Backports is

[Openerp-community-reviewer] [Bug 1008513] Re: [6.1]document_webdav - pywebdav import error if pywebdav=0.9.8

2014-05-16 Thread Yann Papouin
** Summary changed: - document_webdav - pywebdav import error if pywebdav=0.9.8 + [6.1]document_webdav - pywebdav import error if pywebdav=0.9.8 ** Also affects: ocb-addons Importance: Undecided Status: New ** Changed in: ocb-addons Importance: Undecided = Low ** Changed in:

[Openerp-community-reviewer] [Merge] lp:~agilebg/stock-logistic-flows/adding_picking_line_description_7 into lp:stock-logistic-flows

2014-05-16 Thread Alex Comba - Agile BG
Alex Comba - Agile BG has proposed merging lp:~agilebg/stock-logistic-flows/adding_picking_line_description_7 into lp:stock-logistic-flows. Commit message: [ADD] module picking_line_description Requested reviews: Stock and Logistic Core Editors (stock-logistic-core-editors) For more

Re: [Openerp-community-reviewer] [Merge] lp:~therp-nl/account-invoicing/7.0-account_cash_discount into lp:account-invoicing

2014-05-16 Thread Erwin van der Ploeg (BAS Solutions)
Hi Holger, good work. Question. Is there any integration between this cash discount and payment orders? When i make a payment order on time then discount can be extracted. When im passed my discount days it can't. Curious if this exists or is in planning. Thanks. --

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1

2014-05-16 Thread Yann Papouin
You're right. Fixed! -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error/+merge/219794 Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons/6.1. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1

2014-05-16 Thread Pedro Manuel Baeza
Review: Approve code review Thanks for the changes! Regards. -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error/+merge/219794 Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management

2014-05-16 Thread Leonardo Pistone - camptocamp
The proposal to merge lp:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management has been updated. Status: Needs review = Approved For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management

2014-05-16 Thread Leonardo Pistone - camptocamp
As Maxime's remarks should have been addressed, I proceed with the merge. Thank you -- https://code.launchpad.net/~camptocamp/partner-contact-management/add-base_partner_merge/+merge/189616 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. --

[Openerp-community-reviewer] [Merge] lp:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management

2014-05-16 Thread noreply
The proposal to merge lp:~camptocamp/partner-contact-management/add-base_partner_merge into lp:partner-contact-management has been updated. Status: Approved = Merged For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error into lp:ocb-addons/6.1

2014-05-16 Thread Stefan Rijnhart (Therp)
Review: Approve Thanks, I was aware of the problem and the fix looks good. -- https://code.launchpad.net/~yann-papouin/ocb-addons/6.1-bug-1008513-dav-import-error/+merge/219794 Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma

2014-05-16 Thread Leonardo Pistone - camptocamp
Review: Approve code review ok, thanks -- https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde/+merge/218666 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma

2014-05-16 Thread Lionel Sausin - Numérigraphe
Review: Needs Fixing nitpicking Looks good from code point of view, not tested because I don't use this module. Now for the nitpicking: - multiline docstrings should have the short description on the first line, and the details in the following lines. This lets IDEs show the first line as a

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma

2014-05-16 Thread Romain Deheele - Camptocamp
Thanks Lionel, I updated it. -- https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde/+merge/218666 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma

2014-05-16 Thread Lionel Sausin - Numérigraphe
Review: Approve code review, not test -- https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde/+merge/218666 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl

2014-05-16 Thread Lionel Sausin - Numérigraphe
Lionel Sausin - Numérigraphe has proposed merging lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl. Requested reviews: Romain Deheele - Camptocamp (romaindeheele): code review Loïc Bellier - Numérigraphe (lb-b) Purchase Core Editors

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl

2014-05-16 Thread Romain Deheele - Camptocamp
Review: Needs Fixing code review Hi, Several flake8 suggestions: 7.0-add-purchase_budget_limit/purchase_budget_limit/purchase.py:69:25: E128 continuation line under-indented for visual indent 7.0-add-purchase_budget_limit/purchase_budget_limit/purchase.py:76:30: E128 continuation line

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Acsone
Review: Needs Information code review Hello Pedro, Can you explain the use case where you need to enter time on tasks in other state than done? Domains such as this one may vary from a company to another, but I thought allowing only open tasks was a good default? Thanks! --

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl

2014-05-16 Thread Lionel Sausin - Numérigraphe
PEP8 compliance fixed. -- https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit/+merge/219829 Your team Purchase Core Editors is requested to review the proposed merge of lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl.

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma

2014-05-16 Thread noreply
The proposal to merge lp:~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde into lp:openerp-rma has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma_auto_set_warranty-rde/+merge/218666 --

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Pedro Manuel Baeza
I think now in two possible scenarios: - Bad configured project provokes that initial state of a task is not set. - There is a third person that inputs all times. I'm thinking now that tasks can be also even in done state, because inputation is done afterwards. Do you think there is a better

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Acsone
Review: Needs Fixing In that case I would change de domain in an additional module that is perhaps specific to your use case, because I believe current users of hr_timesheet_task expect to see only open tasks. --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test LGTM -- https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745 Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-05-16 Thread Yannick Vaucher @ Camptocamp
The proposal to merge lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745 --

[Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-05-16 Thread noreply
The proposal to merge lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745 --

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Pedro Manuel Baeza
I have reverted back to original domain. Please review rest of the changes. Regards. -- https://code.launchpad.net/~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp/+merge/219821 Your team HR Core Editors is requested to review the proposed merge of

[Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Pedro Manuel Baeza
The proposal to merge lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet has been updated. Description changed to: [FIX] hr_timesheet_sheet: Fix error if there is no project associated to task. [FIX] hr_timesheet_sheet: Tree view for hr.analytic.timesheet inherited

Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/openerp-reporting-engines/7.0-report_xls-tz_fix into lp:openerp-reporting-engines

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test LGTM Cheers. -- https://code.launchpad.net/~luc-demeyer/openerp-reporting-engines/7.0-report_xls-tz_fix/+merge/219267 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:openerp-reporting-engines. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date into lp:purchase-wkfl

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Needs Fixing Please don't add AUTHOR.txt file and add Contributors in __openerp__.py For who did exaclty what there is the CVS -- https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_delivery_split_date/+merge/211374 Your team Purchase Core Editors is subscribed

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl

2014-05-16 Thread Romain Deheele - Camptocamp
Review: Needs Fixing code review, test Thanks, other details: -l.454 s/dificulties/difficulties -Maybe you could precise that we need to check Analytic accounting for purchases in settings/configuration/Purchases or check it directly during addon install. Otherwise, it seems ok. Romain --

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-addons/7.0-bug-1169074-pricelist-category-depth into lp:ocb-addons

2014-05-16 Thread Yann Papouin
It does not seems to be simple to parameterize this query. Maybe this change could be made in another proposal and this one could be merged now ? -- https://code.launchpad.net/~yann-papouin/ocb-addons/7.0-bug-1169074-pricelist-category-depth/+merge/210163 Your team OpenERP Community Backports is

[Openerp-community-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-16 Thread noreply
The proposal to merge lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530 --

[Openerp-community-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-16 Thread Sylvain LE GAL (GRAP)
The proposal to merge lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools has been updated. Commit Message changed to: [ADD] Regression Tests; [FIX] Uncaught error is raised when the admin user tries to connect with a bad login and his password; [ADD] Test for the fixed

Re: [Openerp-community-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Already merged -- https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/account-financial-tools/7.0-add-account_budget_purchase into lp:account-financial-tools

2014-05-16 Thread Lionel Sausin - Numérigraphe
Lionel Sausin - Numérigraphe has proposed merging lp:~numerigraphe-team/account-financial-tools/7.0-add-account_budget_purchase into lp:account-financial-tools. Requested reviews: Loïc Bellier - Numérigraphe (lb-b) Account Core Editors (account-core-editors) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/purchase-wkfl/pallet-delivery into lp:purchase-wkfl

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Needs Fixing MP Description : '' MP Commit message : '' __openerp__.py 'description': , My remarks : '''?''' Please fix this lack of information about this module -- https://code.launchpad.net/~savoirfairelinux-openerp/purchase-wkfl/pallet-delivery/+merge/216325 Your team Purchase

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit into lp:purchase-wkfl

2014-05-16 Thread Loïc Bellier - Numérigraphe
Review: Approve code review Ok for me too ! -- https://code.launchpad.net/~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit/+merge/219829 Your team Purchase Core Editors is requested to review the proposed merge of lp:~numerigraphe-team/purchase-wkfl/7.0-add-purchase_budget_limit

[Openerp-community-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-16 Thread Yannick Vaucher @ Camptocamp
The proposal to merge lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530 --

Re: [Openerp-community-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Please next time explain what you do in your commit message. Fix bug XY is not really understandable at a first glance. To link the bug --fixes=lp:XY is there -- https://code.launchpad.net/~sylvain-legal/server-env-tools/7.0-fix-1319391/+merge/219530 Your team Server Environment And Tools Core

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp into lp:hr-timesheet

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test LGTM -- https://code.launchpad.net/~pedro.baeza/hr-timesheet/7.0-hr_timesheet_task-imp/+merge/219821 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:hr-timesheet. -- Mailing list: https://launchpad.net/~openerp-community-reviewer

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/account-financial-tools/7.0-add-account_budget_purchase into lp:account-financial-tools

2014-05-16 Thread Loïc Bellier - Numérigraphe
Review: Approve code review -- https://code.launchpad.net/~numerigraphe-team/account-financial-tools/7.0-add-account_budget_purchase/+merge/219849 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~bruno-bottacini/report-print-send/7.0-report_webkit_custom_paper_size into lp:report-print-send

2014-05-16 Thread Yannick Vaucher @ Camptocamp
Review: Needs Information Wouldn't it be cleaner to add a hook in report_webkit to create the command ? -- https://code.launchpad.net/~bruno-bottacini/report-print-send/7.0-report_webkit_custom_paper_size/+merge/202892 Your team Report Printing and Sending Core Editors is subscribed to branch

  1   2   >