[ADD] 12.0-connector_magento_multi_source_inventory - #18
Conversation
|
@StefanRijnhart Please have a look. This is WIP. Importer is working. I could not get the exporter working. If you run the tests you will see where it fails. I don't get why it fails - which is the worst thing. the whole connector_magento_multi_source_inventory/models/magento_source/stock_exporter.py file is copied and changes exporter for the stock picking or move - the example you sent. Almost all code needs to be removed except for the run(). |
| the Magento ID, the ID of the Magento Backend and the additional | ||
| fields belonging to the Magento instance. | ||
| """ | ||
| _name = 'magento.binder' |
There was a problem hiding this comment.
I'm pretty sure you are not supposed to list all known apply_ons below. Would it work if you leave out _name here, inherit from 'magento.binder' instead and just set _apply_on = 'magento.source'?
|
|
||
| @api.multi | ||
| def synchronize_metadata(self): | ||
| # super(MagentoBackend, self).synchronize_metadata() |
There was a problem hiding this comment.
I know this is WIP but when you say tests break, not calling super is not going to help.
| binding.name) | ||
|
|
||
| self._validate(binding) | ||
| self._check_allowed_carrier(binding, external_id) |
There was a problem hiding this comment.
Maybe you could remove all this carrier related stuff first, to make it more clear (I can imagine this also makes the tests fail).
|
Stefan, just a heads up. I stopped developing this. I may do some work later when I there is no load in 8. |
|
Agreed, that's why I have already merged the custom module instead. |
|
Closing PR - too old |
2 participants
No description provided.