search

freifunk / cmap.api.freifunk.net

Freifunk Community Map
1 stars 2 forks source link

cleanup and reorganisation #1

Closed mojoaxel closed 11 years ago

mojoaxel commented 11 years ago

Sees anybody any reason not to merge this?

andibraeu commented 11 years ago

ist diese Änderung im anderen Pull Request mit enthalten?

mojoaxel commented 11 years ago

@andibraeu Hier sind nur umstrukturierungen aber keine neuen Features enthalten. Gern etwas mehr im Detail:

Ich mach mir keine Sorgen, das du/Ihr nicht mit der neuen Struktur klar kommt - ist Alles recht ähnlich wie vorher und schön übersichtlich. Ganz klar bekommt das Projekt durch diese Commits etwas meinen Stempel aufgedrückt aber das ist nun mal unausweichlich wenn man sich beteiligt. Ich hatte eigentlich schon bei dem Relaunch von freifunk.net vor die Community-Map zu basteln musste @monic2001 dann aber absagen, da ich erst jetzt die Zeit dafür habe. Gerne kann ich mich auch in Zukunft verstärkt um diese Map kümmern - Hab noch einige Ideen.

Gerne werde ich auch weiterhin Pull-Request nicht gleich automatische mergen sondern erst eure Kommentare abwarten...Außerdem werde ich versuchen die einzelnen Features besser als einzelne Pull-Requests zu verpacken ;-)

andibraeu commented 11 years ago

Ich mach mir keine Sorgen, das du/Ihr nicht mit der neuen Struktur klar kommt - ist Alles recht ähnlich wie vorher und schön übersichtlich. Ganz klar bekommt das Projekt durch diese Commits etwas meinen Stempel aufgedrückt aber das ist nun mal unausweichlich wenn man sich beteiligt.

ja, da hab ich auch keine Befürchtungen :-) Ich habe tatsächlich interessehalber nachgefragt, was da in den Commits passiert ist und wollte mich nicht mühsam durch den Quellcode arbeiten. Außerdem wollte ich herausfinden, ob es in der Benutzung nun zu Unterschieden kommt oder ob wir die Einbindung weiter wie bisher vornehmen können.

Hast du vielleicht irgendwo ne Demo der neuen Karte laufen? oder soll ich sie wieder auf weimarnetz.de installieren?

Gerne kann ich mich auch in Zukunft verstärkt um diese Map kümmern - Hab noch einige Ideen.

das freut mich, wenn du dich hier engagieren willst...

Gerne werde ich auch weiterhin Pull-Request nicht gleich automatische mergen sondern erst eure Kommentare abwarten...Außerdem werde ich versuchen die einzelnen Features besser als einzelne Pull-Requests zu verpacken ;-)

das ist vor allem wichtig, wenn es um Änderungen an der Darstellung oder Einbindung geht. Wir treffen uns am Wochenende in Berlin, um Webthemen zu besprechen und umzusetzen. Vielleicht bauen wir dann auch gleich die neue Karte ein

andibraeu commented 11 years ago

blöderweise bekam ich auch keine Email über einen neuen Pullrequest, wohl weil wir die Repos aufgeteilt haben :-)

mojoaxel commented 11 years ago

Zur Einbindung am besten in Zukunft die map.html als iFrame einbinden. Ein Beispiel gibt es in der neuen index.html. Eine aktuelle Version läuft hier: http://mojoaxel.github.io/cmap.api.freifunk.net/map.html