Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Freifunk Stormarn Community #13

Closed
wants to merge 2 commits into from
Closed

Add Freifunk Stormarn Community #13

wants to merge 2 commits into from

Conversation

kst87
Copy link

@kst87 kst87 commented Jul 7, 2015

No description provided.

kst87 added 2 commits July 6, 2015 20:53
URL hinzugefügt für Freifunk Stormarn
@kst87 kst87 changed the title Domain add Add Freifunk Stormarn Community Jul 7, 2015
@kst87
Copy link
Author

kst87 commented Jul 7, 2015

Würde mich sehr über die Aufnahme der Stormarner Community freuen :)

@AppWerft
Copy link
Owner

AppWerft commented Jul 7, 2015

Dazu bräuchte ich eine URL mit den NodeDaten.

Rainer

Am 07/07/15 um 20:17 schrieb kst87:

Würde mich sehr über die Aufnahme der Stormarner Community freuen :)


Reply to this email directly or view it on GitHub
#13 (comment).

Dipl.-Ing. Rainer Schleevoigt
Certified TYPO3 Integrator
Certified Titanium App Developer

Novalisweg 10

22303 Hamburg
℡ 040 60812460
|Ust-ID: DE239491976|||||||

@kst87
Copy link
Author

kst87 commented Jul 7, 2015

Moin, Name: Stormarn
URL: http://map.ffod.org/data/nodes.json

@AppWerft
Copy link
Owner

AppWerft commented Jul 7, 2015

ist drin.
Rainer

Am 07/07/15 um 20:54 schrieb kst87:

Moin, Name: Stormarn
URL: http://map.ffod.org/data/nodes.json


Reply to this email directly or view it on GitHub
#13 (comment).

Dipl.-Ing. Rainer Schleevoigt
Certified TYPO3 Integrator
Certified Titanium App Developer

Novalisweg 10

22303 Hamburg
℡ 040 60812460
|Ust-ID: DE239491976|||||||

@reimersjan
Copy link

@AppWerft Du kannst die Pull-Requests auch einfach mergen. Da stehen die URLs ja drin. Und die Pull-Requester tauchen bei Contributors auf, was ja auch nett ist um zu sehen wer was zur App beigetragen hat.

@AppWerft
Copy link
Owner

AppWerft commented Jul 8, 2015

keien Ahnung, wo ich da klicken muss.

Am 07/07/15 um 22:49 schrieb Jan Reimers:

@AppWerft https://github.com/AppWerft Du kannst die Pull-Requests
auch einfach mergen. Da stehen die URLs ja drin
https://github.com/AppWerft/Freifunker/pull/13/files. Und die
/Pull-Requester/ tauchen bei Contributors
https://github.com/AppWerft/Freifunker/graphs/contributors auf, was
ja auch nett ist um zu sehen wer was zur App beigetragen hat.


Reply to this email directly or view it on GitHub
#13 (comment).

Dipl.-Ing. Rainer Schleevoigt
Certified TYPO3 Integrator
Certified Titanium App Developer

Novalisweg 10

22303 Hamburg
℡ 040 60812460
|Ust-ID: DE239491976|||||||

@reimersjan
Copy link

@AppWerft Direkt im Pull-Request erscheint diese Box, wenn der Code direkt übernommen werden kann und es keine Konflikte gibt:
mergepr Screenshot

Bei diesem Pull-Request (in dem wir uns gerade befinden) steht dort nun This pull request contains merge conflicts that must be resolved weil du den Code wahrscheinlich kopiert hast und selber eingefügt hast.

Hier ein Einblick in die Funktionsweise von Pull-Requests https://www.youtube.com/watch?v=d5wpJ5VimSU

Allgemein ist es eher unmotivierend für Menschen die etwas zu einem Projekt beitragen wollen, wenn Pull-Requests einfach ohne Diskussion oder Grund geschlossen werden. Das war mir in diesem Repository schon ein paar mal aufgefallen. Besser ist es, die Pull-Requests nach Prüfung zu mergen oder im Kommentarbereich des Pull-Requests zu diskutieren, warum der Code nicht (oder noch nicht) ins Repository übernommen werden kann, anstatt den Pull-Request kommentarlos zu schließen.

Ich hoffe ich konnte ein wenig Klarheit verschaffen.

@AppWerft
Copy link
Owner

AppWerft commented Jul 8, 2015

wir sollten uns mal zu einer Gitsesssion treffen. Bin oft im TRANSMONTANA.

Am 08/07/15 um 12:19 schrieb Jan Reimers:

@AppWerft https://github.com/AppWerft Direkt im Pull-Request
erscheint diese Box, wenn der Code direkt übernommen werden kann und
es keine Konflikte gibt:
mergepr Screenshot
https://cloud.githubusercontent.com/assets/4367193/8568886/42b49b62-2572-11e5-919b-f816101bda8c.png

Bei diesem Pull-Request (in dem wir uns gerade befinden) steht dort
nun This pull request contains merge conflicts that must be resolved
weil du den Code wahrscheinlich kopiert hast und selber eingefügt hast.

Hier ein Einblick in die Funktionsweise von Pull-Requests
https://www.youtube.com/watch?v=d5wpJ5VimSU

Allgemein ist es eher unmotivierend für Menschen die etwas zu einem
Projekt beitragen wollen, wenn Pull-Requests einfach ohne Diskussion
oder Grund geschlossen werden. Das war mir in diesem Repository schon
ein paar mal aufgefallen
https://github.com/AppWerft/Freifunker/pulls?q=is%3Apr+is%3Aclosed.
Besser ist es, die Pull-Requests nach Prüfung zu mergen oder im
Kommentarbereich des Pull-Requests zu diskutieren, warum der Code
nicht (oder noch nicht) ins Repository übernommen werden kann, anstatt
den Pull-Request kommentarlos zu schließen.

Ich hoffe ich konnte ein wenig Klarheit verschaffen.


Reply to this email directly or view it on GitHub
#13 (comment).

Dipl.-Ing. Rainer Schleevoigt
Certified TYPO3 Integrator
Certified Titanium App Developer

Novalisweg 10

22303 Hamburg
℡ 040 60812460
|Ust-ID: DE239491976|||||||

@AppWerft
Copy link
Owner

URL funzt nicht. htttps://map.ffod.org/data/nodes.json

@AppWerft AppWerft closed this Jan 30, 2016
@kst87
Copy link
Author

kst87 commented Jan 30, 2016

Moin, jo...url hat sich geändert.
Neu: http://map.stormarn.freifunk.net/data/nodes.json

@AppWerft
Copy link
Owner

Habe ich soeben eingebaut.

@kst87
Copy link
Author

kst87 commented Jan 30, 2016

Bedankt

@AppWerft
Copy link
Owner

dauert jetzt noch ca. 1 Stunde.

@AppWerft
Copy link
Owner

Am 30/01/16 um 11:52 schrieb Kaj:

Bedankt


Reply to this email directly or view it on GitHub
#13 (comment).

Dipl.-Ing. Rainer Schleevoigt
Certified TYPO3 Integrator
Certified Titanium App Developer

Novalisweg 10

22303 Hamburg
℡ 040 60812460
|Ust-ID: DE239491976|||||||

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants