[Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-11 Thread noreply
The proposal to merge lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management has been updated. Status: Needs review = Merged For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-11 Thread Guewen Baconnier @ Camptocamp
Review: Approve code review -- https://code.launchpad.net/~akretion-team/partner-contact-management/partner-firstname-inheritable/+merge/222730 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-11 Thread Pedro Manuel Baeza
Review: Approve code review Hi, Etienne, Thanks for the new MP. Although I wonder why are you still modifying priority on stock.move.tree3, I'm not going to block the MP for this question, because I don't think that this alters behaviour on possible modules relying on that view. Regards. --

Re: [Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-11 Thread Etienne Hirt
Dear Pedro, Thanks for the approval. As you mentioned correctly not to remove stock.move.tree3 in a stable version, I wanted to make clear that this view is less important by changing the priority. But I agree it has no effect. BestRegards Etienne On 11.06.2014 08:18, Pedro Manuel Baeza wrote:

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/server-env-tools/module-configuration-helper-dbl into lp:server-env-tools

2014-06-11 Thread Florian da Costa
Review: Approve test I now use it for delivery_carrier_chronopost and it work well! Thanks David -- https://code.launchpad.net/~akretion-team/server-env-tools/module-configuration-helper-dbl/+merge/220392 Your team Server Environment And Tools Core Editors is subscribed to branch

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into lp:openerp-product-attributes

2014-06-11 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into lp:openerp-product-attributes. Requested reviews: Product Core Editors (product-core-editors) For more details, see:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/7.0-send-message-slow-1311087 into lp:ocb-addons

2014-06-11 Thread Guewen Baconnier @ Camptocamp
Guewen Baconnier @ Camptocamp has proposed merging lp:~camptocamp/ocb-addons/7.0-send-message-slow-1311087 into lp:ocb-addons. Requested reviews: OpenERP Community Backports (ocb) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into lp:openerp-product-attributes

2014-06-11 Thread Alexis de Lattre
Review: Approve code review and test Alexandre, could you replay my commit revno 249 in the branch lp:~akretion-team/openerp-product-attributes/70-update-and-fix-product-gtin ; it's a small code simplification. --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe into lp:openerp-product-attributes

2014-06-11 Thread Alexis de Lattre
Review: Approve code review and test BTW, for those who want to make EAN8 work in the POS, here is the patch for odoo master : http://people.via.ecp.fr/~alexis/ean8-pos-works.diff -- https://code.launchpad.net/~camptocamp/openerp-product-attributes/7.0-add_product_gtin-afe/+merge/222760 Your

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread Acsone
Review: Approve code review Makes sense to me. -- https://code.launchpad.net/~camptocamp/account-financial-report/7.0-account_export_csv-select-periods/+merge/222806 Your team Account Report Core Editors is requested to review the proposed merge of

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread Romain Deheele - Camptocamp
Review: Approve code review LGTM too. -- https://code.launchpad.net/~camptocamp/account-financial-report/7.0-account_export_csv-select-periods/+merge/222806 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread Guewen Baconnier @ Camptocamp
Guewen Baconnier @ Camptocamp has proposed merging lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report. Requested reviews: Account Report Core Editors (account-report-core-editor) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread Matthieu Dietrich @ camptocamp
Review: Approve LGTM, and makes sense. -- https://code.launchpad.net/~camptocamp/account-financial-report/7.0-account_export_csv-select-periods/+merge/222806 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread Nicolas Bessi - Camptocamp
Review: Approve LGTM -- https://code.launchpad.net/~camptocamp/account-financial-report/7.0-account_export_csv-select-periods/+merge/222806 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report

2014-06-11 Thread noreply
The proposal to merge lp:~camptocamp/account-financial-report/7.0-account_export_csv-select-periods into lp:account-financial-report has been updated. Status: Needs review = Merged For more details, see:

[Openerp-community-reviewer] [Bug 1311004] Re: missing database index on account_move_line(date, id)

2014-06-11 Thread Olivier Dony (OpenERP)
Merged in 7.0 on github at https://github.com/odoo/odoo/commit/5d77de54e5c1b7465d133e508f8986b8249e0368. Thanks! ** Changed in: openobject-addons Importance: Undecided = Wishlist ** Changed in: openobject-addons Status: New = Fix Released -- You received this bug notification because

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-location into lp:stock-logistic-warehouse

2014-06-11 Thread Lionel Sausin - Numérigraphe
Loïc and I are still working on this proposal. Good progress was made: we re-aligned the features with v8 in mind, and improved the code style and the views. Acsone joined in and we're still making tests, and i hope Loïc can make a new proposal in the coming days. --

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical into lp:stock-logistic-warehouse

2014-06-11 Thread Lionel Sausin - Numérigraphe
Review: Resubmit test pending Loïc and I are still working on this proposal. Good progress was made: we re-aligned the features with v8 in mind, and improved the code style and the views. Acsone joined in and we're still making tests, and I hope Loïc can make a new proposal in the coming days.

[Openerp-community-reviewer] [Merge] lp:~andrei-levin/ocb-addons/6.1 into lp:ocb-addons/6.1

2014-06-11 Thread Andrei Levin
Andrei Levin has proposed merging lp:~andrei-levin/ocb-addons/6.1 into lp:ocb-addons/6.1. Requested reviews: OpenERP Community Backports (ocb) For more details, see: https://code.launchpad.net/~andrei-levin/ocb-addons/6.1/+merge/222862 Timestamps in body part of the meeting invitation email