Re: [Openerp-community-reviewer] [Question #257730]: Redmine - Odoo connector

2014-11-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Question #257730 on Redmine - OpenERP Connector changed: https://answers.launchpad.net/redmine-openerp/+question/257730 Status: Open = Answered Maxime Chambreuil (http://www.savoirfairelinux.com) proposed the following answer: Hello Boujraf, There is no plan yet. -- You received

Re: [Openerp-community] pull request reviews

2014-08-06 Thread Maxime Chambreuil
are talking about a pull request merged the same day after 2 reviews here and introducing code bypassing the PEP8 compliance check. I am sure even Pedro will agree that it does not allow too much time for others to review. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original

Re: [Openerp-community] pull request reviews

2014-08-06 Thread Maxime Chambreuil
Pedro, We are based in Montreal, Canada and could have reviewed it. It's day time for us when it's night for you. Like anyone, you don't see much when it's late at night ;) Ping us next time and we could be your third review. Thanks. -- Maxime Chambreuil +1 (514) 276-5468 #126

Re: [Openerp-community] [Merge] lp:~openerp-community/openerp-canada/7.0-missing-payroll-access into lp:openerp-canada

2014-07-09 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~openerp-community/openerp-canada/7.0-missing-payroll-access/+merge/226187 Your team OpenERP Community is subscribed to branch lp:~openerp-community/openerp-canada/7.0-missing-payroll-access. ___ Mailing

Re: [Openerp-community] [Merge] lp:~openerp-community/openerp-canada/7.0-missing-payroll-access into lp:openerp-canada

2014-06-30 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing account is not a dependency of this module. Use hr groups instead. -- https://code.launchpad.net/~openerp-community/openerp-canada/7.0-missing-payroll-access/+merge/225031 Your team OpenERP Community is subscribed to branch

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-06-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260 Your team OpenERP Community Reviewer/Maintainer is requested to review the proposed merge of lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0.

Re: [Openerp-community] Documentation V8

2014-06-17 Thread Maxime Chambreuil
+1 for technical documentation -1 for functional one -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - +1 El 17/06/2014 06:16, João Almeida jpoalme...@gmail.com escribió: +1 On Tue, Jun 17, 2014 at 5:41 AM, David Arnold - El Alemán da...@elaleman.co

Re: [Openerp-community] Documentation V8

2014-06-17 Thread Maxime Chambreuil
I don't dislike functional documentation. Functional persons should not have to learn git to produce documentation, that's it. -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Not sure why you would dislike functional documentation? +1 for finally some more

[Openerp-community] Result of the OCA Financial Auditors

2014-06-16 Thread Maxime Chambreuil
to receive your credentials to perform your duties this week. Thank you very much to all candidates and delegates. Next election will be organized next year. Have a great day. -- Maxime Chambreuil +1 (514) 276-5468 #126 ___ Mailing list: https

Re: [Openerp-community] [Merge] lp:~openerp-community/openerp-canada/7.0-missing-payroll-access into lp:openerp-canada

2014-06-11 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~openerp-community/openerp-canada/7.0-missing-payroll-access/+merge/222815 Your team OpenERP Community is subscribed to branch lp:~openerp-community/openerp-canada/7.0-missing-payroll-access. ___ Mailing

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-reviewer] [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-reviewer] [Merge] lp:~sylvain-legal/server-env-tools/7.0-fix-1319391 into lp:server-env-tools

2014-05-14 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- 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] [Merge] lp:~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing into lp:openerp-canada

2014-05-06 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing into lp:openerp-canada has been updated. Status: Merged = Approved For more details, see:

[Openerp-community-reviewer] Deprecated modules in 8.0

2014-04-26 Thread Maxime Chambreuil
* project_long_term to project-service Please let me know if you think of better places. Cheers -- Maxime Chambreuil +1 (514) 276-5468 #126 -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-community-reviewer@lists.launchpad.net Unsubscribe

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension into lp:openerp-manufacturing

2014-04-26 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension/+merge/211696 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:openerp-manufacturing. -- Mailing list: https://launchpad.net/~openerp-community-reviewer

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing test @ehdem l205: Can you put CMIS versions (1.0 or 1.1, not sure) ? 1.7 is a Magento version. Thanks -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing test I got a stacktrace instead of an error message when testing the connection: Server Traceback (most recent call last): File /home/max/openerp/cmis/bzr/web/addons/web/session.py, line 89, in send return openerp.netsvc.dispatch_rpc(service_name, method, args) File

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
test -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:knowledge-addons/7.0. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-04-24 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
test -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:knowledge-addons/7.0. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community] [Merge] lp:~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing into lp:openerp-canada

2014-04-22 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing/+merge/216708 Your team OpenERP Community is subscribed to branch lp:~openerp-community/openerp-canada/7.0-l10n_ca_account_check_writing.

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis/+merge/212258 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:knowledge-addons/7.0. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis_write into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_write/+merge/213940 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:knowledge-addons/7.0. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/knowledge-addons/cmis_read into lp:knowledge-addons/7.0

2014-04-19 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/knowledge-addons/cmis_read/+merge/212260 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:knowledge-addons/7.0. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

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

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
I just fixed the branch according to your reviews. Thanks. -- https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl. Requested reviews: Sale Core Editors (sale-core-editors) For more details, see: https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking into lp:sale-wkfl has been updated. Description changed to: [ADD] sale_lot_tracking For more details, see: https://code.launchpad.net/~savoirfairelinux-openerp/sale-wkfl/sale_lot_tracking/+merge/216323 --

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

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast into lp:purchase-wkfl. Commit message: [ADD] Add purchase profit forecast Requested reviews: Purchase Core Editors (purchase-core-editors) For more

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge into lp:account-invoicing

2014-04-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/account-invoicing/account_invoice_line_merge into lp:account-invoicing. Commit message: [ADD] account_invoice_purchase_origin module Requested reviews: Account Core Editors (account-core

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/lp-community-utils/checkout-pep8 into lp:lp-community-utils

2014-03-17 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/checkout-pep8/+merge/211098 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:lp-community-utils. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to

[Openerp-community-reviewer] Sylvain LE GAL application to OpenERP Community Reviewer / Maintainer

2014-03-14 Thread Maxime Chambreuil
Hi, I would like to belong to this group. Is it possible ? I'm currently active in the following project : - https://launchpad.net/web-addons - https://launchpad.net/server-env-tools Regards. -- This message was sent from Launchpad by Sylvain LE GAL (GRAP)

Re: [Openerp-community-reviewer] Sylvain LE GAL application to OpenERP Community Reviewer / Maintainer

2014-03-14 Thread Maxime Chambreuil
Yes, you missed the subject of the email ;) It is Sylvain Le Gal application, not mine. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Hi Maxime, You're in through the Reviewer group right ? Or do I miss something ? Regards, Joël On Fri

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/server-env-tools/l10n into lp:server-env-tools

2014-03-14 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/server-env-tools/l10n/+merge/20 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 :

Re: [Openerp-community] OpenERP Marketing

2014-02-15 Thread Maxime Chambreuil
OpenERP answers a wide range of needs, so you need to differentiate the positioning based on the 3 axis above. In each of those axis, the competition is different and arguments toward OpenERP will be different as well. My 2 cents. -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406 into lp:openerp-canada

2014-02-13 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1213406 into lp:openerp-canada. Requested reviews: OpenERP Community Reviewer/Maintainer (openerp-community-reviewer) Related bugs: Bug #1213406 in OpenERP Canada

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/lp-community-utils/progressbar into lp:lp-community-utils

2014-02-06 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/progressbar/+merge/205252 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:lp-community-utils. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community] OpenERP New Website: Community Page

2014-02-04 Thread Maxime Chambreuil
events Join * Your local team * OpenERP Community Association Contribute * Translations (Launchpad links, link to guidelines) * Functional expertise (mailing lists) * Code Review (mailing list, Community guideline in doc.openerp.com) -- Maxime Chambreuil +1

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/lp-community-utils/pep394 into lp:lp-community-utils

2014-02-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~savoirfairelinux-openerp/lp-community-utils/pep394/+merge/204535 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:lp-community-utils. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community] CMIS connector

2014-01-28 Thread Maxime Chambreuil
I guess so. It looks like the appropriate library to use. -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Very interesting, Maxime, Do you plan to use the Apache Chemistry Python cmislib? Regards, Brice 2014-01-28 Maxime Chambreuil maxime.chambre

Re: [Openerp-community] CMIS connector

2014-01-28 Thread Maxime Chambreuil
I just filled in their contact form. I will let you know if there is something available to test and where it is. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Hello, I think it is in production somewhere... call syleam, the code should be here

Re: [Openerp-community] Partner firstname

2014-01-23 Thread Maxime Chambreuil
text/html; charset=utf-8: Unrecognized attachment: image/jpeg___ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community More help :

[Openerp-community] Partner firstname

2014-01-22 Thread Maxime Chambreuil
://blueprints.launchpad.net/partner-contact-management/+spec/partner-firstname Feel free to join . -- Maxime Chambreuil +1 (514) 276-5468 #126 ___ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@lists.launchpad.net Unsubscribe

Re: [Openerp-community] Fwd: Partner firstname

2014-01-22 Thread Maxime Chambreuil
I am not sure everyone would be interested by mother's name and surname. We are not. What do you think of keeping those 2 fields in a separate module with a dependency on partner_firstname ? -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Hi Maxime, may

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/partner-contact-management/bug-1269036 into lp:partner-contact-management

2014-01-16 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
I don't know if it is technically possible, but could we use the name field to store the lastname value and have name+firstname displayed in a relationship ? -- https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/bug-1269036/+merge/201947 Your team Partner and Contact

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

2014-01-07 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
+++ base_external_dbsource/__openerp__.py 2014-01-07 22:26:44 + @@ -0,0 +1,61 @@ +# -*- coding: utf-8 -*- +## +# +#Daniel Reis, 2011 +#Additional contributions by Maxime Chambreuil, Savoir-faire Linux

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

2014-01-07 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools. Requested reviews: Server Environment And Tools Core Editors (server-env-tools-core-editors) Nicolas Bessi

Re: [Openerp-connector-community] Multi-company and update stock level

2014-01-03 Thread Maxime Chambreuil
text/html; charset=utf-8: Unrecognized attachment: elico_signature.jpg-- Mailing list: https://launchpad.net/~openerp-connector-community Post to : openerp-connector-community@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-connector-community More help :

Re: [Openerp-community-reviewer] [Merge] lp:~initos.com/account-analytic/7.0 into lp:account-analytic

2014-01-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Hello Markus, I don't know if it has an impact but the PO files still contains the previous name of the module. Otherwise I would approve it. Thanks. -- https://code.launchpad.net/~initos.com/account-analytic/7.0/+merge/195362 Your team Account Core Editors

Re: [Openerp-community-reviewer] [Merge] lp:~initos.com/account-analytic/7.0 into lp:account-analytic

2014-01-03 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~initos.com/account-analytic/7.0/+merge/195362 Your team Account Core Editors is subscribed to branch lp:account-analytic. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2014-01-02 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-invoice-report. -- Mailing list:

[Openerp-community-reviewer] [Blueprint resume-management] Resume Management

2013-12-31 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Blueprint changed by Maxime Chambreuil (http://www.savoirfairelinux.com): Definition Status: New = Approved -- Resume Management https://blueprints.launchpad.net/openerp-hr/+spec/resume-management -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067 into lp:openerp-fiscal-rules

2013-12-30 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/openerp-fiscal-rules/7.0-fix-bug-1265067 into lp:openerp-fiscal-rules. Commit message: [MRG] Fix _fiscal_position_map() got multiple values for keyword argument 'uid' Requested reviews

Re: [Openerp-community-reviewer] [Merge] lp:~serpentcs/server-env-tools/7.0-base_synchro into lp:server-env-tools

2013-12-28 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Hi GRaeme, Shall we reject this MP ? -- https://code.launchpad.net/~serpentcs/server-env-tools/7.0-base_synchro/+merge/183102 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list: https://launchpad.net/~openerp-community-reviewer

[Openerp-community] [Merge] lp:~openerp-community/openerp-product-variant/fix-pep8-and-relative-import into lp:~anybox/openerp-product-variant/7.0

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~openerp-community/openerp-product-variant/fix-pep8-and-relative-import into lp:~anybox/openerp-product-variant/7.0 has been updated. Status: Needs review = Work in progress For more details, see:

Re: [Openerp-community] [Merge] lp:~vauxoo/openerp-product-attributes/7.0-prod_vol_weight-dev-humbertoarocha into lp:openerp-product-attributes

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review @hbto Here are what the lines should be : l30: from . import model l122: from . import product l43 to l45: Can you move those lines in the module description ? To answer Stefan, can you add the UOM in the help of the field (l162) ? --

[Openerp-community] [Merge] lp:~vauxoo/openerp-product-attributes/7.0-prod_vol_weight-dev-humbertoarocha into lp:openerp-product-attributes

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~vauxoo/openerp-product-attributes/7.0-prod_vol_weight-dev-humbertoarocha into lp:openerp-product-attributes has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe into lp:stock-logistic-flows/7.0

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~camptocamp/stock-logistic-flows/7.0-add-stock_recompute_availability_on_force-afe into lp:stock-logistic-flows/7.0 has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~parthiv-patel/web-addons/web-addons into lp:web-addons

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~parthiv-patel/web-addons/web-addons into lp:web-addons has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~parthiv-patel/web-addons/web-addons/+merge/179846 --

[Openerp-community-reviewer] [Merge] lp:~lin-yu/account-invoicing/add_account_invoce_line into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~lin-yu/account-invoicing/add_account_invoce_line into lp:account-invoicing has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~lin-yu/account-invoicing/add_account_invoce_line/+merge/182531 --

[Openerp-community-reviewer] [Merge] lp:~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix into lp:ocb-addons

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix into lp:ocb-addons has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~florian-dacosta/ocb-addons/7.0_purchase_line_partner_fix/+merge/185331 --

[Openerp-community-reviewer] [Merge] lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account into lp:account-invoicing has been updated. Status: Needs review = Work in progress For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~bcim/account-invoicing/7.0-account_invoice_line_default_account into lp:account-invoicing

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review l282,283: can you add from . import... ? -- https://code.launchpad.net/~bcim/account-invoicing/7.0-account_invoice_line_default_account/+merge/186078 Your team Account Core Editors is subscribed to branch lp:account-invoicing. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~akretion-team/account-financial-tools/70-fiscal-position-check-vat into lp:account-financial-tools

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~akretion-team/account-financial-tools/70-fiscal-position-check-vat into lp:account-financial-tools has been updated. Status: Needs review = Work in progress For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/report-print-send/7.0-base_report_to_printer_port-imp-yvr-2 into lp:report-print-send

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~camptocamp/report-print-send/7.0-base_report_to_printer_port-imp-yvr-2/+merge/188825 Your team Report Printing and Sending Core Editors is subscribed to branch lp:report-print-send. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Hello Sébastien, Few changes : l21: from . import ... l140: Move AUTHORS content in the module description. l166: from . import ... You need to add the pot files for both modules. --

[Openerp-community-reviewer] [Merge] lp:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~sebastien.beau/stock-logistic-flows/stock-logistic-flows-61-stock_picking_better_invoicing into lp:stock-logistic-flows has been updated. Status: Needs review = Work in progress For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~acsone-openerp/ocb-web/active_domain-7.0-backport into lp:ocb-web

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Abstain -- https://code.launchpad.net/~acsone-openerp/ocb-web/active_domain-7.0-backport/+merge/189279 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-web. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

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

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Please fix your PEP8 issues: $ flake8 base_partner_merge/ --ignore=E501,F401 base_partner_merge/base_partner_merge.py:47:1: F812 list comprehension redefines 'email' from line 39 base_partner_merge/base_partner_merge.py:66:1: E302 expected 2 blank lines, found

[Openerp-community-reviewer] [Merge] lp:~serpentcs/account-financial-report/account-serpentcs into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/account-financial-report/account-serpentcs into lp:account-financial-report has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~serpentcs/account-financial-report/account-serpentcs/+merge/190320 --

Re: [Openerp-community-reviewer] [Merge] lp:~therp-nl/ocb-addons/6.1_lp1237832 into lp:ocb-addons/6.1

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~therp-nl/ocb-addons/6.1_lp1237832/+merge/190312 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~serpentcs/purchase-report/purchase-serpentcs into lp:purchase-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/purchase-report/purchase-serpentcs into lp:purchase-report has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~serpentcs/purchase-report/purchase-serpentcs/+merge/190323 --

Re: [Openerp-community-reviewer] [Merge] lp:~serpentcs/sale-reports/sale-serpentcs into lp:sale-reports

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Hello Nishant, Can you use local import (from . import ...) in your __init__.py files ? I am not familiar with webkit report. Don't they need a translation file ? Thanks. -- https://code.launchpad.net/~serpentcs/sale-reports/sale-serpentcs/+merge/190321 Your

[Openerp-community-reviewer] [Merge] lp:~serpentcs/webkit-utils/utils-serpentcs into lp:webkit-utils

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/webkit-utils/utils-serpentcs into lp:webkit-utils has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~serpentcs/webkit-utils/utils-serpentcs/+merge/190327 --

Re: [Openerp-community-reviewer] [Merge] lp:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Hello Lorenzo, Please remove the AUTHORS.txt file and add its content to the module description. Thanks. -- https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230 Your team OpenERP Community

[Openerp-community-reviewer] [Merge] lp:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~agilebg/account-invoice-report/adding_account_invoice_production_lot_7 into lp:account-invoice-report has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into lp:openerp-sage-50

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50 into lp:openerp-sage-50 has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~savoirfairelinux-openerp/openerp-sage-50/7.0-exportsage50/+merge/191893

[Openerp-community-reviewer] [Merge] lp:~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~vauxoo/account-financial-report/7.0-report_webkit_afr-_data_dev-jorge into lp:account-financial-report has been updated. Status: Needs review = Work in progress For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls/+merge/192250 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-tools/7.0-account_pain into lp:account-financial-tools

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~luc-demeyer/account-financial-tools/7.0-account_pain into lp:account-financial-tools has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~luc-demeyer/account-financial-tools/7.0-account_pain/+merge/192256 --

Re: [Openerp-community-reviewer] [Merge] lp:~pedro.baeza/stock-logistic-flows/7.0-stock_picking_show_returns into lp:stock-logistic-flows/7.0

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~pedro.baeza/stock-logistic-flows/7.0-stock_picking_show_returns/+merge/193075 Your team Stock and Logistic Core Editors is subscribed to branch lp:stock-logistic-flows/7.0. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review Please fix the conflicts. -- https://code.launchpad.net/~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations/+merge/200119 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report. -- Mailing

[Openerp-community-reviewer] [Merge] lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report

2013-12-27 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Maxime Chambreuil (http://www.savoirfairelinux.com) has proposed merging lp:~erwin-bassolutions/account-financial-report/7.0-report-webkit-afr-dutch-translations into lp:account-financial-report. Requested reviews: Pedro Manuel Baeza (pedro.baeza): code review, no test Guewen Baconnier

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into lp:openerp-hr/6.1

2013-12-23 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review and test -- https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data/+merge/194763 Your team HR Core Editors is requested to review the proposed merge of lp:~savoirfairelinux-openerp/openerp-hr/6.1-fix-hr-skill-demo-data into

Re: [Openerp-community-reviewer] [Merge] lp:~therp-nl/server-env-tools/7.0-my_OpenERP_is_not_unsupported into lp:server-env-tools

2013-12-21 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve -- https://code.launchpad.net/~therp-nl/server-env-tools/7.0-my_OpenERP_is_not_unsupported/+merge/199896 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] [Merge] lp:~savoirfairelinux-openerp/openerp-product-attributes/product_dependencies into lp:openerp-product-attributes

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~savoirfairelinux-openerp/openerp-product-attributes/product_dependencies into lp:openerp-product-attributes has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~lin-yu/purchase-wkfl/add_purchase_control_suuplier into lp:purchase-wkfl

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~lin-yu/purchase-wkfl/add_purchase_control_suuplier into lp:purchase-wkfl has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~lin-yu/purchase-wkfl/add_purchase_control_suuplier/+merge/180795 --

[Openerp-community-reviewer] [Merge] lp:~jean-lelievre/purchase-report/product_stock_type into lp:purchase-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/purchase-report/product_stock_type into lp:purchase-report has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~jean-lelievre/purchase-report/product_stock_type/+merge/182050 --

[Openerp-community-reviewer] [Merge] lp:~jean-lelievre/purchase-report/move_reports into lp:purchase-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/purchase-report/move_reports into lp:purchase-report has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~jean-lelievre/purchase-report/move_reports/+merge/182044 --

[Openerp-community-reviewer] [Merge] lp:~jean-lelievre/stock-logistic-report/stock_batch_move into lp:stock-logistic-report

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~jean-lelievre/stock-logistic-report/stock_batch_move into lp:stock-logistic-report has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~jean-lelievre/stock-logistic-report/stock_batch_move/+merge/182060 --

[Openerp-community-reviewer] [Merge] lp:~pedro.baeza/account-financial-report/6.1-balance-reporting into lp:account-financial-report/6.1

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~pedro.baeza/account-financial-report/6.1-balance-reporting into lp:account-financial-report/6.1 has been updated. Status: Needs review = Work in progress For more details, see:

[Openerp-community-reviewer] [Merge] lp:~serpentcs/server-env-tools/7.0-base_synchro into lp:server-env-tools

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
The proposal to merge lp:~serpentcs/server-env-tools/7.0-base_synchro into lp:server-env-tools has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~serpentcs/server-env-tools/7.0-base_synchro/+merge/183102 --

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

2013-12-20 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Thanks Yannick. I made the changes according to your review. -- https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/183539 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list:

Re: [Openerp-community] [Openerp-community-reviewer] Review day

2013-12-16 Thread Maxime Chambreuil
://test.savoirfairelinux.com/job/OCA/73/console We could run it every thursday night and get the email sent to the community list. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Dear community, I just receive a mail from Alexis De Lattre concerning the rules

Re: [Openerp-community-reviewer] Review day

2013-12-16 Thread Maxime Chambreuil
://test.savoirfairelinux.com/job/OCA/73/console We could run it every thursday night and get the email sent to the community list. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Dear community, I just receive a mail from Alexis De Lattre concerning the rules

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis

2013-12-12 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge/+merge/198551 Your team Margin Analysis Core Editors is requested to review the proposed merge of lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/partner-contact-management/city-move into lp:partner-contact-management

2013-12-12 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Don't forget where you are from. There is no state in France ;) -- https://code.launchpad.net/~savoirfairelinux-openerp/partner-contact-management/city-move/+merge/196023 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/margin-analysis/7.0-cost-price-store-improv-jge into lp:margin-analysis

2013-12-11 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Needs Fixing code review and test The new field Cost price is on the Information tab in the product form. It would be better on the Procurements one, with the standard price field. I have tested the calculation of the cost price based on components cost price and the inventory

Re: [Openerp-community-reviewer] OpenERP

2013-12-11 Thread Maxime Chambreuil
I also support Yannick's application to the reviewer team. Regards, -- Maxime Chambreuil +1 (514) 276-5468 #126 - Mail original - Dear all, I'm voting in favor of Sandy Carter and Yann Papouin as member of the reviewer team. I'm also suggesting Yannick Vaucher as he

Re: [Openerp-community-reviewer] [Merge] lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension into lp:openerp-hr

2013-12-11 Thread Maxime Chambreuil (http://www.savoirfairelinux.com)
Review: Approve code review -- https://code.launchpad.net/~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension/+merge/198620 Your team HR Core Editors is requested to review the proposed merge of lp:~savoirfairelinux-openerp/openerp-hr/hr_employee_phone_extension into

  1   2   3   >