Merge lp://staging/~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15 into lp://staging/magentoerpconnect/oerp6.0-stable
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15 |
Merge into: | lp://staging/magentoerpconnect/oerp6.0-stable |
Diff against target: |
332 lines (+257/-7) 5 files modified
magentoerpconnect/product.py (+5/-3) magentoerpconnect/product_view.xml (+1/-0) magentoerpconnect/settings/external.mapping.template.csv (+13/-0) magentoerpconnect/settings/external.mappinglines.template.csv (+237/-4) magentoerpconnect/settings/external.referential.type.csv (+1/-0) |
To merge this branch: | bzr merge lp://staging/~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sébastien BEAU - http://www.akretion.com | Needs Fixing | ||
Review via email: mp+67429@code.staging.launchpad.net |
Description of the change
[IMP] For Magento 1.5
To post a comment you must log in.
Unmerged revisions
- 428. By Maxime Chambreuil (http://www.savoirfairelinux.com)
-
[FIX]
- 427. By Maxime Chambreuil (http://www.savoirfairelinux.com)
-
[IMP] Import Eric McSween changes on openerp5-module to openerp6-module
Hello Maxime sorry for the delay, we are very busy here.
For the merge, I talk with Raphael and finally we decide to change the organization of the mapping.
Indeed if all mapping (for each version) are in the same file, it's going to be very difficult to update it, to fix it, and also the visibility is going to dicrease.
It's why we propose to use a new structure.
In setting we will have one folder per mapping version and so if we need to fix a mapping in all of the version, or to see the different between two version it will be very easy (for example with meld we can compare two file easily).
http:// bazaar. launchpad. net/~magentoerp connect- core-editors/ magentoerpconne ct/trunk_ version/ files/head: /magentoerpconn ect/settings/
What do you think? Can you port the fix on the mapping on the trunk version and propose a merge?
(for trunk version note that in the mapping all "str" have been replaced by "unicode")
Have a nice day and thanks for your interest on the connector