openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~akretion-team/partner-contact-management/base-location-geonames-import into lp:partner-contact-management

To: Alexis de Lattre <alexis@xxxxxxxxxx>
From: Alexis de Lattre <alexis@xxxxxxxxxx>
Date: Fri, 11 Apr 2014 21:33:18 -0000
In-reply-to: <20140407151921.18727.35936.launchpad@ackee.canonical.com>
Reply-to: mp+214564@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

@Nicolas:

I confirm that it returns 404 if you try to download a wrong country.

I have taken into account your other remarks in my last commits (unless the tests ; I don't have experience in this). I am not sure I fully understand the benefit of the FOR UPDATE NOWAIT in this particular case... at least I hope I implemented it as you wanted.
-- 
https://code.launchpad.net/~akretion-team/partner-contact-management/base-location-geonames-import/+merge/214564
Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management.

References

[Merge] lp:~akretion-team/partner-contact-management/base-location-geonames-import into lp:partner-contact-management
From: Alexis de Lattre, 2014-04-07