Code review comment for lp:~vthompson/ubuntu-clock-app/clock-location

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Hi Victor, Thanks for the patch. I swear when I implemented this I discussed extensively with Martin Borho about which variables to use and I vaguely remember him explaining that adminName2 and adminName3 worked well in German cities where it is common for duplicate names to appear. Let me check with him and also try out different locations to see if "name" returns better results than "adminName2".

« Back to merge proposal