-
-
Notifications
You must be signed in to change notification settings - Fork 209
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
base: 8.0
Are you sure you want to change the base?
Conversation
Based on the work on OCA#43.
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
Thanks 👍 |
* vals: field values of the new record, e.g {'field_name': field_value, ...} | ||
""" | ||
|
||
on_product_write = Event() |
There was a problem hiding this comment.
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
Is someone working on this? |
yes me 😄 |
Great :) Before going deep on the PR, is it fully functional? |
@liebana This one is working for one of our customers. |
@damdam-s does it support simple (one attribute - multiple values) variants ? |
No description provided.