Great thanks

On Wed, Jun 20, 2012 at 3:50 PM, susmit shannigrahi <thinklinux.ssh@gmail.com> wrote:
On Wed, Jun 20, 2012 at 1:53 AM, Ankur Sinha <sanjay.ankur@gmail.com> wrote:
> On Tue, 2012-06-19 at 20:39 +0800, Mathieu Bridon wrote:
>> I'm pretty sure patches are, as usual, welcome:
>>     https://fedorahosted.org/fama/browser/ambassadors_map
>
> I played around with the code and well.. it works ;)
>
> - the data file[1] needs to be regenerated using the script[2] (it has
> an .erb extension, I moved it to correct this to .py)
> Please make the following change so that it asks you for your FAS and
> password, rather than using the dummy:
>
>
>>     #username = 'fedoradummy'
>>     #password = '<%= fedoraDummyUserPassword %>'
>>     username = raw_input('Username: ').strip()
>>     password = getpass.getpass('Password: ')

Dummy and .erb are for hosting the script.
These are fine.

> - The ambassadors.html[3] file worked for me after this tiny change
> (could be an openlayers API change). Patch here[5]:
>
>
>> 47c47
>> <             Osm = new OpenLayers.Layer.OSM.Osmarender("Open Street Map");
>> ---
>> >             Osm = new OpenLayers.Layer.OSM("Open Street Map");
>> 50,51c50,51
>> <             Mapnik = new OpenLayers.Layer.OSM.Mapnik("Mapnik");
>> <             map.addLayer(Mapnik);
>> ---
>> > //            Mapnik = new OpenLayers.Layer.OSM.Mapnik("Mapnik");
>> > //            map.addLayer(Mapnik);

Ah, thanks for the patch.

> I've hosted a working version here[4]
>
> Please ensure that your locations in FAS are up to date.
>
> Joerg, Susmit: I think you folks take care of this. I'd request that
> this map be re-generated on a quarterly basis (the last change to the
> data was made 3 years ago according to the trac[6]). I could take care
> of it if required. I don't know how to work server side etc., but I'm
> sure I can learn :)

No, the data is generated every day. It is not three years old.

[susmit@susmit ~]$ wget -S
http://fedoraproject.org/membership-map/ambassadors_location.txt
[susmit@susmit ~]$ ls ambassadors_location.txt  -l
-rw-rw-r--. 1 susmit susmit 45442 Jun 19 23:52 ambassadors_location.txt

I will fix the map.
Thanks.
--
Regards,
Susmit.

=============================================
http://www.fedoraproject.org/wiki/user:susmit
=============================================



--
Ardian Haxha
Fedora Ambassador
Fedora Project


http://www.twitter.com/ArdianHaxha
http://www.identi.ca/ardoam
irc.freenode.net #flossk #fedora #fedora-ambassadors