Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Magentoerpconnect catalog simple2 #200

Open
wants to merge 31 commits into
base: 8.0
Choose a base branch
from

Conversation

damdam-s
Copy link
Member

No description provided.

sebastienbeau and others added 30 commits March 11, 2016 13:14
These new classes do nothing by default and will be 'replaced' in the
'magentoerpconnect_catalog_simple' module which will import new fields.
[IMP] import product_attribute_set from magento
Because the number of classes tend to grow fast in connector addons, organize
directories using a hierarchy:

- models
|-- model_name
|---- common.py: hold Models, Binders, Adapters (what is shared for importers and exporters)
|---- importer.py: hold functions and classes related to imports
|---- exporter.py: hold functions and classes related to exports
[IMP] show attribute_set_id field in module catalog_simple
@guewen
Copy link
Member

guewen commented Mar 11, 2016

Thanks 👍

* vals: field values of the new record, e.g {'field_name': field_value, ...}
"""

on_product_write = Event()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those 2 ought to be added in connector_base_product

@liebana
Copy link

liebana commented Jul 7, 2016

Is someone working on this?

@damdam-s
Copy link
Member Author

damdam-s commented Jul 7, 2016

yes me 😄

@liebana
Copy link

liebana commented Jul 7, 2016

Great :) Before going deep on the PR, is it fully functional?

@damdam-s
Copy link
Member Author

damdam-s commented Jul 8, 2016

@liebana This one is working for one of our customers.
Note that it doesn't support multi-variant.

@wpichler
Copy link

@damdam-s does it support simple (one attribute - multiple values) variants ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants