Mathis Kirchner sneeex
sneeex automatically merged pull request liquid-development/game-tracker#67 2025-12-10 20:43:54 +00:00
sneeex commented on pull request liquid-development/game-tracker#67 2025-12-10 20:37:44 +00:00
CreateGameView erstellen

so vom logischen

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-10 20:37:39 +00:00
CreateGameView erstellen

ja ich weiß aber macht es nicht mehr sinn das ausserhalb zu machen

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-10 20:11:09 +00:00
CreateGameView erstellen

und warum hast du den futurebuilder entfernt?

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-10 20:10:46 +00:00
CreateGameView erstellen

macht das nicht mehr sinn, rein logisch, die ausserhalb der future wait zu haben? weil die hat ja nichts in der future wait zu suchen oder?

sneeex suggested changes for liquid-development/game-tracker#67 2025-12-10 20:10:46 +00:00
CreateGameView erstellen
sneeex suggested changes for liquid-development/game-tracker#67 2025-12-10 12:27:31 +00:00
CreateGameView erstellen

chef also wirklich guckst du auch selber nach fehlern?

sneeex suggested changes for liquid-development/game-tracker#67 2025-12-10 08:23:50 +00:00
CreateGameView erstellen

Also bitte teste doch bevor du zur Review gibst einmal die Functionality und suche nach eventuellen Bugs, folgendes Problem:

sneeex suggested changes for liquid-development/game-tracker#67 2025-12-09 21:32:04 +00:00
CreateGameView erstellen

Player werden nach auswählen der Gruppe nicht sortiert

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-09 20:18:10 +00:00
CreateGameView erstellen

bitte mit listener implementieren, außerdem hat das _gameNameController hier keinen Effekt, würde auch nur mit setState und leerem Inhalt gehen

sneeex suggested changes for liquid-development/game-tracker#67 2025-12-09 20:18:10 +00:00
CreateGameView erstellen
sneeex commented on pull request liquid-development/game-tracker#67 2025-12-09 08:26:18 +00:00
CreateGameView erstellen

Nicht mergen

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-08 16:52:32 +00:00
CreateGameView erstellen

Obwohl andere Sache: gehört #87 nicht auch hierzu? Hat ja wahrscheinlich damit zu tun, dass du nach dem Aufrufen vom Navigator mit dem du die CreateGameView zeigst nicht reloadest

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-08 16:49:00 +00:00
CreateGameView erstellen

ist mir auch egal, lass es so oder nicht. Wenn du es mit Info/Error im Titel willst, mach am besten das einmal temporär in den Titel und dann nen Issue für Änderung der anderen Titel in eine enum…

sneeex commented on pull request liquid-development/game-tracker#67 2025-12-08 16:46:44 +00:00
CreateGameView erstellen

ja findste mit oder ohne richtigen Title besser?

sneeex opened issue liquid-development/game-tracker#87 2025-12-08 13:43:56 +00:00
Winner aktualisiert beim setzen direkt nach Game Erstellung nicht die GameHistoryView
sneeex commented on pull request liquid-development/game-tracker#67 2025-12-08 13:39:55 +00:00
CreateGameView erstellen

Hier bitte ohne Punkt, das hatten wir ja gesagt, und wir müssen uns darauf einigen ob der Titel infos erhält, oder nur keywords wie info/error, weil ich das bis jetzt so gemacht habe.

sneeex suggested changes for liquid-development/game-tracker#67 2025-12-08 13:39:55 +00:00
CreateGameView erstellen
sneeex commented on pull request liquid-development/game-tracker#67 2025-12-07 20:55:11 +00:00
CreateGameView erstellen

Frage: ist es beabsichtigt, dass wenn ich spieler auswähle, dann ne group auswähle und dann wieder zurück gehe die Spieler nicht mehr da sind und die Seite neu lädt?

Also eigentlich…