← Back to team overview

openerp-canada team mailing list archive

Re: lp:~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15 into lp:magentoerpconnect/openerp6-module

 

Review: Needs Fixing

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/~magentoerpconnect-core-editors/magentoerpconnect/trunk_version/files/head:/magentoerpconnect/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
-- 
https://code.launchpad.net/~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15/+merge/67429
Your team OpenERP Canada Team is subscribed to branch lp:~magentoerpconnect-community/magentoerpconnect/openerp6-module-for-magento15.


References