Image upload error handling

[blacksenator]

Improvements if something went wrong during image upload

[andig]

Bitte Space vor der Klammer entfernen

[andig]

Feature branches sollten immer neu von master geforked werden

[blacksenator]

Oh, ich dachte das hätte ich - sorry. Ich denke beim nächsten dran Volker Püschel Am Wolfshagen 4 30974 Wennigsen T +49-5045-911-484 M +49-177-7960362 F +49-5045-911-500 [email protected]

…

Am 26.12.2018 um 23:55 schrieb andig ***@***.***>: Feature branches sollten immer neu von master geforked werden — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[blacksenator]

Hallo Andreas, noch mal für mich zum Verständnis: Bisher habe ich das so gemacht: 1. ich bin lokal in TortoiseGit in meinen master branch gewechselt und habe dann einen feature branch erzeugt 2. In den feature branch gewechselt und programmiert 3. commit und PUSH des feature branch auf mein remote (origin) 4. wechsel in GitHub (hab in TortoiseGit keinen Pendant-Funktion gefunden) in den neuen feature branch und PULL REQUEST zu deinem master branch 5. abwarten, code review, ... 6. nach deinem merge habe ich in TortoiseGit deinen master branch (upstream) per PULL auf meinen master branch lokal gezogen 7. danach habe ich per PUSH meinen master lokal auf meinen master remote (origin) gepusht 8. dann wieder wie bei 1. Stimmt der Ablauf oder was mache ich falsch? Denn eigentlich ... ...müssten jetzt dein master, und meine master remote und lokal gleich sein!? Trotzdem steht in GitHub: "This branch is 6 commits ahead of andig:master." und die Commits dazu irritieren mich auch weil unterschiedlich. Irgendwas mach ich falsch oder habe es in der Vergangenheit doch abweichend anders gemacht als oben beschrieben...  Volker Am 26.12.2018 um 23:58 schrieb Volker Püschel:

…

Oh, ich dachte das hätte ich - sorry. Ich denke beim nächsten dran Volker Püschel Am Wolfshagen 4 30974 Wennigsen T +49-5045-911-484 M +49-177-7960362 F +49-5045-911-500 ***@***.*** Am 26.12.2018 um 23:55 schrieb andig ***@***.*** ***@***.***>>: > Feature branches sollten immer neu von master geforked werden > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#68 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AeBdzntwYdFuL_aiBU6l8BuQmMTaYbmmks5u8_5sgaJpZM4Zh9_4>. >

[andig]

Der Ablauf klingt richtig. Aber- hier ist der Vergleich: master...BlackSenator:master <master...BlackSenator:master> und hier Deine Commits: https://github.com/BlackSenator/carddav2fb/commits/master <https://github.com/BlackSenator/carddav2fb/commits/master> Der Fehler liegt anscheinend in Schritt 6: Du machst einen Merge statt einfach ein Pull? Ich mach das auf der Kommandozeile: git pull upstream master Die Remote in dem Fall heisst einfach upstream (im Gegensatz zu origin). Wie Du den Merge da fabrizierst ist mir nicht klar, stört aber auch nicht außer dass es komisch aussieht. Und die Commits kommen dann natürlich in jedem PR wieder mit… Wenn Dus ändern willst geht das zerstörerisch so: git reset --hard <sha1-commit-id> dann einfach wie oben nachdem Du upstream als meinen Fork hinzugefügt hast. Viele Grüße, Andreas

…

On 27. Dec 2018, at 09:26, Black Senator ***@***.***> wrote: Hallo Andreas, noch mal für mich zum Verständnis: Bisher habe ich das so gemacht: 1. ich bin lokal in TortoiseGit in meinen master branch gewechselt und habe dann einen feature branch erzeugt 2. In den feature branch gewechselt und programmiert 3. commit und PUSH des feature branch auf mein remote (origin) 4. wechsel in GitHub (hab in TortoiseGit keinen Pendant-Funktion gefunden) in den neuen feature branch und PULL REQUEST zu deinem master branch 5. abwarten, code review, ... 6. nach deinem merge habe ich in TortoiseGit deinen master branch (upstream) per PULL auf meinen master branch lokal gezogen 7. danach habe ich per PUSH meinen master lokal auf meinen master remote (origin) gepusht 8. dann wieder wie bei 1. Stimmt der Ablauf oder was mache ich falsch? Denn eigentlich ... ...müssten jetzt dein master, und meine master remote und lokal gleich sein!? Trotzdem steht in GitHub: "This branch is 6 commits ahead of andig:master." und die Commits dazu irritieren mich auch weil unterschiedlich. Irgendwas mach ich falsch oder habe es in der Vergangenheit doch abweichend anders gemacht als oben beschrieben... Volker Am 26.12.2018 um 23:58 schrieb Volker Püschel: > Oh, > > ich dachte das hätte ich - sorry. Ich denke beim nächsten dran > > Volker Püschel > Am Wolfshagen 4 > 30974 Wennigsen > T +49-5045-911-484 > M +49-177-7960362 > F +49-5045-911-500 ***@***.*** > > Am 26.12.2018 um 23:55 schrieb andig ***@***.*** > ***@***.***>>: > >> Feature branches sollten immer neu von master geforked werden >> >> — >> You are receiving this because you authored the thread. >> Reply to this email directly, view it on GitHub >> <#68 (comment)>, >> or mute the thread >> <https://github.com/notifications/unsubscribe-auth/AeBdzntwYdFuL_aiBU6l8BuQmMTaYbmmks5u8_5sgaJpZM4Zh9_4>. >> — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#68 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AALR7-6N51FjMuoVqkLH2_mXmmV06uOqks5u9IQzgaJpZM4Zh9_4>.