search

andig / carddav2fb

Download CardDAV VCards and upload as phonebook to AVM FRITZ!Box
63 stars 19 forks source link

Image upload error handling #68

Closed blacksenator closed 5 years ago

blacksenator commented 5 years ago

Improvements if something went wrong during image upload

andig commented 5 years ago

Feature branches sollten immer neu von master geforked werden

blacksenator commented 5 years ago

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 volker.pueschel@anasco.de

Am 26.12.2018 um 23:55 schrieb andig notifications@github.com:

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 commented 5 years ago

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 volker.pueschel@anasco.de

Am 26.12.2018 um 23:55 schrieb andig <notifications@github.com mailto:notifications@github.com>:

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 https://github.com/andig/carddav2fb/pull/68#issuecomment-450037855, or mute the thread https://github.com/notifications/unsubscribe-auth/AeBdzntwYdFuL_aiBU6l8BuQmMTaYbmmks5u8_5sgaJpZM4Zh9_4.

andig commented 5 years ago

Der Ablauf klingt richtig. Aber- hier ist der Vergleich:

https://github.com/andig/carddav2fb/compare/master...BlackSenator:master https://github.com/andig/carddav2fb/compare/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

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 notifications@github.com 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 volker.pueschel@anasco.de

Am 26.12.2018 um 23:55 schrieb andig <notifications@github.com mailto:notifications@github.com>:

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 https://github.com/andig/carddav2fb/pull/68#issuecomment-450037855, 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 https://github.com/andig/carddav2fb/pull/68#issuecomment-450102648, or mute the thread https://github.com/notifications/unsubscribe-auth/AALR7-6N51FjMuoVqkLH2_mXmmV06uOqks5u9IQzgaJpZM4Zh9_4.