search

cozy / cozy-photos-v2

Deprecated - New : https://github.com/cozy/cozy-drive/tree/master/src/photos - Personal Photo Gallery Manager
http://cozy.io
GNU Affero General Public License v3.0
27 stars 25 forks source link

Development #150

Closed miton18 closed 8 years ago

miton18 commented 8 years ago

add feature to cozy-photos: photos geolocalisation on map

clochix commented 8 years ago

Wow, thanks @miton18, that's huge ! Well review this soon, stay tuned ☺

aenario commented 8 years ago

Ok, this PR is awesome ! I just tried it on my local instance and its flawless.

I made a couple comment on code style and async logic (There is a few things which might cause problems at 10000 pictures).

Let me know if you have any question. Thank you very much !

miton18 commented 8 years ago

Vous voulez que je corrige le code et que je refasse un commit? ou vous gérez les modifs?

clochix commented 8 years ago

Bonjour @miton18 et merci pour cette contribution ! Oui, le processus classique est que l'auteur de la PR fait lui-même les corrections. Donc fais un commit avec les remarques de Romain, il s'ajoutera à cette PR et si tout est bon nous pourrons la fusionner ☺

miton18 commented 8 years ago

Es-ce que tout est bon? ^^

clochix commented 8 years ago

Merci beaucoup @miton18 ☺ @aenario va jeter un dernier œil et ça devrait être bon.

aenario commented 8 years ago

Salut @miton18 , yep, tout est bon ! Merci beaucoup