Code review comment for lp:~ajite/openerp-connector/7.0-magentoerpconnect-fix-000002

Revision history for this message
Augustin Cisterne-Kaas - www.elico-corp.com (ajite) wrote :

> On 08/19/2013 09:51 AM, Guewen Baconnier @ Camptocamp wrote:
> > Anyhow, I need a bit more reflexion on that, but my feeling is that for
> > such cases, we should never deactivate the exports and just use the
> > "visibility" or "active" field of Magento.
> >
> That would also prevent many issues with cross-linking between records,
> e.g. export a child category but parent category's not yet exported, or
> export a product linked (cross-sell, up-sell, ...) to a product not yet
> exported.
>
>
> --
> Guewen Baconnier
> Business Solutions Software Developer
>
> Camptocamp SA
> PSE A, CH-1015 Lausanne
> Phone: +41 21 619 10 39
> Office: +41 21 619 10 10
> http://www.camptocamp.com/

You are absolutely right.
I have removed the global "push on save" option. I will try to think about something else.

« Back to merge proposal