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

2014-01-08 Thread Alexandre Fayolle - camptocamp
Review: Approve code review, no test -- 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-report/7.0-account_move_line_report_xls into lp:account-financial-report

2014-01-08 Thread Alexandre Fayolle - camptocamp
The proposal to merge lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report has been updated. Status: Needs review = Approved For more details, see:

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

2014-01-08 Thread noreply
The proposal to merge lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report has been updated. Status: Approved = Merged 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

2014-01-08 Thread Alexis de Lattre
@Alexandre : Thanks for the merge... but it doesn't contain the new list of default fields we decided above in the function _report_xls_fields() in account_move_line.py. Could you update it ? --

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

2014-01-08 Thread Frederic Clementi - Camptocamp
@ Alexandre @ Alexis Oups Sorry my mistake. I told Alexandre it was ok. -- 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

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

2014-01-08 Thread Frederic Clementi - Camptocamp
@Luc I agree with the new default list as well -- 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:

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

2014-01-08 Thread Luc De Meyer (Noviat)
Don't worry. I'll make a new merge with the default fields and documentation update. Luc -Original Message- From: boun...@canonical.com [mailto:boun...@canonical.com] On Behalf Of Alexis de Lattre Sent: woensdag 8 januari 2014 10:21 To: mp+192...@code.launchpad.net Subject: Re:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-humanitarian-ngo/add-bid_selected_state_on_po-nbi into lp:openerp-humanitarian-ngo/purchase-wkfl

2014-01-08 Thread Nicolas Bessi - Camptocamp
Add a bid selected state on PO Fix confirmation of cost estimate. PO in state bid will be cancel too -- https://code.launchpad.net/~camptocamp/openerp-humanitarian-ngo/add-bid_selected_state_on_po-nbi/+merge/200799 Your team OpenERP for Humanitarian Core Editors is requested to review the

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-humanitarian-ngo/purchase-wkfl-fix_confirm-nbi into lp:openerp-humanitarian-ngo/purchase-wkfl

2014-01-08 Thread Nicolas Bessi - Camptocamp
The proposal to merge lp:~camptocamp/openerp-humanitarian-ngo/purchase-wkfl-fix_confirm-nbi into lp:openerp-humanitarian-ngo/purchase-wkfl has been updated. Status: Needs review = Rejected For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-humanitarian-ngo/purchase-wkfl-fix_confirm-nbi into lp:openerp-humanitarian-ngo/purchase-wkfl

2014-01-08 Thread Nicolas Bessi - Camptocamp
Review: Disapprove Deprecated by https://code.launchpad.net/~camptocamp/openerp-humanitarian-ngo/add-bid_selected_state_on_po-nbi/+merge/200799 -- https://code.launchpad.net/~camptocamp/openerp-humanitarian-ngo/purchase-wkfl-fix_confirm-nbi/+merge/197918 Your team OpenERP for Humanitarian Core

Re: [Openerp-community-reviewer] [Merge] lp:~yann-papouin/ocb-web/6.1-bug-1261322-useful-report-filename into lp:ocb-web/6.1

2014-01-08 Thread Holger Brunn (Therp)
I'd say action['name'] while taking care of filesystem unfriendly strings there -- https://code.launchpad.net/~yann-papouin/ocb-web/6.1-bug-1261322-useful-report-filename/+merge/27 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-web/6.1. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~elicoidal/multi-company/multi-company into lp:multi-company

2014-01-08 Thread Acsone
Hello Eric, We are investigating possibilities for a customer that has multicompany supply chain issues. Your blueprint on the topic looks very promising and we would like to investigate further if this client could contribute some testing and/or development. What is the status of the

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report

2014-01-08 Thread Nicolas Bessi - Camptocamp
Hello, Pedro idea is not bad but it goes against readability and lead to future error. This is more a core openerp design flaw/bug, and it should be reported as a server bug (if not allready done). It should be handeld by the tree graphing behavior (or the poor man's solution an helper)

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report

2014-01-08 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report. Requested reviews: Account Report Core Editors (account-report-core-editor) Related bugs: Bug #1266701 in Account -

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/contract-management/7.0-block-send-email-lep into lp:contract-management

2014-01-08 Thread Leonardo Pistone @ camptocamp
Pedro, I added the pot file. Best -- https://code.launchpad.net/~camptocamp/contract-management/7.0-block-send-email-lep/+merge/200812 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:contract-management. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/6.1_backport_account_export_csv_company_fix into lp:account-financial-report/6.1

2014-01-08 Thread Matthieu Dietrich @ camptocamp
Review: Approve Okay, managed to (finally) resubmit this correctly. -- https://code.launchpad.net/~camptocamp/account-financial-report/6.1_backport_account_export_csv_company_fix/+merge/200836 Your team Account Report Core Editors is subscribed to branch lp:account-financial-report/6.1. --

Re: [Openerp-community-reviewer] [Merge] lp:~elicoidal/multi-company/multi-company into lp:multi-company

2014-01-08 Thread Eric Caudal - www.elico-corp.com
Hi Stéphane, Our new module is quite stable now and fully tested. It relies now on the new connector (magento-connector) to improve job queue management It allows (with complex multi-directional communication): SO = PO PO = SO SO = SO (new) Soon to be added: DO = DO with partial quantity

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report

2014-01-08 Thread Guewen Baconnier @ Camptocamp
Review: Approve code review Thanks -- https://code.launchpad.net/~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe/+merge/200810 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-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report

2014-01-08 Thread Guewen Baconnier @ Camptocamp
The proposal to merge lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report has been updated. Status: Needs review = Approved For more details, see:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report

2014-01-08 Thread noreply
The proposal to merge lp:~camptocamp/account-financial-report/7.0-dont_crash_if_report_xls_not_available-afe into lp:account-financial-report has been updated. Status: Approved = Merged For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~mikel-martin/purchase-wkfl/6.1-purchase_landed_costs into lp:purchase-wkfl/6.1

2014-01-08 Thread mikel
Yet another try. Does anyone know why eclipse doesn't complain about E121 and E127? I've configured eclipse to use pep8 in this way http://stackoverflow.com/questions/399956/how-to-integrate-pep8-py-in-eclipse/8532188#8532188 --