Comment 2 for bug 1183837

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Hi Brendan,

No worries about the branch, I'm already pleased to see your bug report and patch.

For a next time, you can just push a branch on :

lp:~brendan-clune/openerp-connector/name-of-your-branch

The permissions will be correct because they depend on the project (second part of the url).

Your patch is correct, I checked in the old mappings and the field is called 'region' whatever was the Magento version. I'll merge it right now.

Thanks,
Guewen