Open MadEyeRoody opened 8 years ago
Sehr gut @MadEyeRoody
Grad gepusht @Pfoerd
Da ich die Back Buttons entfernt habe ist fast alles betroffe.
pushed @Pfoerd änderungen beziehen sich auf den Graphen (rot Fall versucht dazustellen)
pushed @Pfoerd
button bar in empfehlungen ist jetzt footer, scanner symbol auf startseite habe ich geändert
pushed @Pfoerd
durch mergekonflikte gingen wieder 2 stunden arbeit verloren
Hey Rudi, mist, aber wie das? Der Merge müsste doch eigentlich trivial gewesen sein, ich habe in der controller.js nur auf der Einstiegsseite hantiert und du meintest du wärst im Finanzstatus mit den Charts unterwegs???
Gruß Ferdi
Hi Ferdi,
lag hauptsächlich an der geilen Mergelogik im Webstorm. Der war anderer meinung als ich ;) Hatte gar nicht im Kopf, dass du gerade auch pushed. Passe in Zukunft weiter auf, konnte aber den Code retten, indem ich eine apk außeinandergenommen hab.
Hey, ok alles klar. Dann mal was anderes seit ich mir deine aktuellste Änderung reingeholt habe funktiniert die Finanzstatus-Seite bei mir nicht mehr, kommt eine Javascript-Fehler:
Ist das ok im Moment?
Kann ich jetzt nicht nachvollziehen. Bei mir läuft alles. Hab gerade nochmal gepusht. (schau mal auf die Info beim gelben Smiley und sag mir was du ca davon hälst.
Ok offensichtlich geht es im Firefox nicht mehr.... Im Chrome geht es noch. Mit welchem Browser entwickelst du?
ich entwickle eigtl in beidem. Die Grafik ist des einzige was überall läuft ^^
Habe gepusht. Ich hoffe jetzt ist alles wieder gut :\
Hey Rudi. Arbeitest du heute Vormittag an der App? Wenn nicht würde ich mal unsere Controller.js in einzelne Files (ein Controller je File) aufspliten, damit wir uns nicht so häufig ins Gehege kommen.
Gruß Ferdi
hab grad nochmal gemerged und wieder gepusht. Teil des Zeug auf und gib bescheid wenn du fertig bist :-)
Ist drin. Ich hoffe die Zusammenarbeit wird dadurch etwas leichter :) .
Schau doch bitte mal ob alles von dir drin ist (insbesondere vom letzten Push von dir).
dann bekomm ichs mit wenn du pusht und kann pullen/ mergen ohne zuviel aufwand :-) @Pfoerd