search

brownbaglunch / BrownBagLunch

www.brownbaglunch.fr/
Other
58 stars 168 forks source link

added bagger: adrien joly #305

Closed adrienjoly closed 9 years ago

fsarradin commented 9 years ago

Bienvenue Adrien !

adrienjoly commented 9 years ago

Merci François!

Par contre j'obtiens l'erreur suivante en allant sur http://www.brownbaglunch.fr/baggers.html:

Uncaught TypeError: Cannot read property 'baggers' of undefined

J'espère que c'est pas mon pull request qui a causé ça! :-s

++

On Mon, May 11, 2015 at 11:18 AM, François Sarradin < notifications@github.com> wrote:

Bienvenue Adrien !

— Reply to this email directly or view it on GitHub https://github.com/nrichand/BrownBagLunch/pull/305#issuecomment-100829522 .

Adrien Joly adrienjoly.com

fsarradin commented 9 years ago

Vu ! En fait, ça provient indirectement de ton pull request : Montreuil n'est pas une ville référencée dans la liste des villes qui se trouve un peu plus bas dans le fichier bagger.js. Du coup, on a deux solutions: 1/ ajouter Montreuil dans cette liste 2/ retirer Montreuil de ta liste citites,

Sachant les listes cities sont plus de la meta-donnée destinée en particulier à l'affichage dans Google Map, je propose qu'on parte sur la solution #2, Qu'en penses-tu ?

nrichand commented 9 years ago

C'est ce que j'étais en train de pusher ;) Bien vu @fsarradin

adrienjoly commented 9 years ago

ok pas de problème pour la solution deux.

souhaites tu que je fasse a nouveau un pull request pour intégrer cette modif?

2015-05-11 11:35 GMT+02:00 François Sarradin notifications@github.com:

Vu ! En fait, ça provient indirectement de ton pull request : Montreuil n'est pas une ville référencée dans la liste des villes qui se trouve un peu plus bas dans le fichier bagger.js. Du coup, on a deux solutions: 1/ ajouter Montreuil dans cette liste 2/ retirer Montreuil de ta liste citites,

Sachant les listes cities sont plus de la meta-donnée destinée en particulier à l'affichage dans Google Map, je propose qu'on parte sur la solution #2 https://github.com/nrichand/BrownBagLunch/pull/2, Qu'en penses-tu ?

— Reply to this email directly or view it on GitHub https://github.com/nrichand/BrownBagLunch/pull/305#issuecomment-100833259 .

Adrien Joly adrienjoly.com

nrichand commented 9 years ago

Non, ne t'inquiète pas je push le fix

fsarradin commented 9 years ago

je croix que @nrichand va s'en occuper ;)

nrichand commented 9 years ago

Si par contre tu veux vraiment rajouter Montreuil comme point sur la carte tu pourras refaire une PR

adrienjoly commented 9 years ago

avec plaisir! par contre il faut aussi que j'upload une photo dans ce cas, c'est bien ça?

je suis a votre écoute s'il y a des guidelines a respecter.

2015-05-11 11:39 GMT+02:00 Nathaniel Richand notifications@github.com:

Si par contre tu veux vraiment rajouter Montreuil comme point sur la carte tu pourras refaire une PR

— Reply to this email directly or view it on GitHub https://github.com/nrichand/BrownBagLunch/pull/305#issuecomment-100833867 .

Adrien Joly adrienjoly.com