connector-magento icon indicating copy to clipboard operation
connector-magento copied to clipboard

8.0 Add support for multi company

Open lmignon opened this issue 8 years ago • 7 comments

The goal of this PR is to add the ability to support multi companies with the connector Magento. This PR follows the same idea as the one expressed in the blueprint of @guewen https://blueprints.launchpad.net/openerp-connector-magento/+spec/import-multi-company multi company imp At this stage multi company is supported for

  • [X] sale.order: the sale.order are created by company
  • [X] res.partner: the re.partner are created without company if the backend is configured in multi company mode.

In our case, we don't need to import the products from magento, therefore, nothing has been modified to support the mutli company on products.

  • [ ] product.product

By default nothing is provided by magento to support different warehouses by website/store/storeview. In our case we use a specific addon for magento (from smile) and we have developped a specialized odoo addon to export the product qty by warehouse.

  • [ ] stock???

lmignon avatar May 20 '16 08:05 lmignon

Hi @lmignon, @laetitia-gangloff, is this perhaps ready for review?

StefanRijnhart avatar Jul 07 '16 07:07 StefanRijnhart

@StefanRijnhart This PR provides a partial solution to support multi companies with magento. This code is used in production. IMO it's ready for review...

lmignon avatar Jul 18 '16 07:07 lmignon

We have been testing this (we can live with the limitations). :+1: and thanks!

StefanRijnhart avatar Jul 18 '16 11:07 StefanRijnhart

@StefanRijnhart Indeed, we only use this information to compute the stock for a specific warehouse into magento (custom module in magento)

lmignon avatar Oct 12 '16 11:10 lmignon

@lmignon How about https://github.com/acsone/connector-magento/pull/1?

StefanRijnhart avatar Oct 12 '16 12:10 StefanRijnhart

@susport those errors are unrelated to this change and can be fixed separately.

StefanRijnhart avatar Mar 08 '17 14:03 StefanRijnhart

@guewen Can you consider this PR in your V10 migration ?

Cedric-Pigeon avatar Jun 14 '17 14:06 Cedric-Pigeon