[Openerp-community-reviewer] [Merge] lp:~anybox/openerp-product-variant/7.0 into lp:openerp-product-variant

2013-10-07 Thread noreply
The proposal to merge lp:~anybox/openerp-product-variant/7.0 into lp:openerp-product-variant has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~anybox/openerp-product-variant/7.0/+merge/183609 --

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

2013-10-07 Thread Yannick Vaucher @ Camptocamp
base_partner_merge is patched to ignore old remaining tables which could still exist in database after migration -- https://code.launchpad.net/~camptocamp/partner-contact-management/add-base_partner_merge/+merge/189616 Your team Partner and Contact Core Editors is requested to review the

Re: [Openerp-community-reviewer] [Merge] lp:~the-clone-master/ocb-addons/6.1-lp212930-purchase_anglo_saxon_invoice_from_PO_lines into lp:ocb-addons/6.1

2013-10-07 Thread Virgil Dupras
Review: Approve I lack the expertise and time to properly review this MP, but I did like Stefan and compared the diff from the trunk with this MP and they look identical except for the lines 16-19 which aren't in the trunk's diff. Now it's up to the maintainer of this project to determine

Re: [Openerp-community-reviewer] [Merge] lp:~elbati/purchase-wkfl/adding_purchase_partial_invoicing_7 into lp:purchase-wkfl

2013-10-07 Thread Lorenzo Battistini - Agile BG
On 10/01/2013 07:30 PM, Pedro Manuel Baeza wrote: Review: Needs Fixing code review and test Hi, Lorenzo, thank you very much for the changes. There's only one more thing that I have detected: if you make a partial invoice, when you make again another invoice, you can let deselected Partial

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

2013-10-07 Thread Stefan Rijnhart (Therp)
This proposal now has some pretty bad conflicts. Can you try to repair? -- https://code.launchpad.net/~vauxoo/server-env-tools/7.0_merge_partner_duplicates/+merge/170122 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing list:

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

2013-10-07 Thread Stefan Rijnhart (Therp)
Review: Needs Information Are you aware that the same feature was backported here: https://code.launchpad.net/~vauxoo/server-env-tools/7.0_merge_partner_duplicates/+merge/170122 (It was in pretty good shape when I reviewed that branch, but something went wrong afterwards) --

Re: [Openerp-community-reviewer] [Merge] lp:~invitu/account-financial-tools/add_account_chart_update_module_for_v70 into lp:account-financial-tools

2013-10-07 Thread Pedro Manuel Baeza
Review: Needs Information Thank you very much for your work, but there has been here an overlapped work, because there was already an MP for this module adaptation a week ago: https://code.launchpad.net/~joaquing-pedrosa/account-financial-tools/7.0-account-chart-update/+merge/188649 We can