Mathis Kirchner sneeex
sneeex commented on pull request liquid-development/game-tracker#67 2025-11-26 16:51:51 +00:00
CreateGameView erstellen

evtl besser gametype und ruleset aufteilen und wenn man dann einen gametype auswählt, wird automatisch ein ruleset ausgewählt, ein gametype ist aber nicht erforderlich, ein ruleset schon. Das…

sneeex commented on pull request liquid-development/game-tracker#76 2025-11-26 16:10:35 +00:00
Fehlende Methoden für Games Datenbank inplementieren

*correctly

sneeex commented on pull request liquid-development/game-tracker#76 2025-11-26 16:10:35 +00:00
Fehlende Methoden für Games Datenbank inplementieren

warum steht hier if a game player or group exists it will be replaced? bezieht sich das auf das game, bzw. die group/den player im game? Vorhin meintest du doch das replace funktioniert nicht so

sneeex commented on pull request liquid-development/game-tracker#76 2025-11-26 16:10:35 +00:00
Fehlende Methoden für Games Datenbank inplementieren

sind hier nicht addGroupToGame und updateGroupOfGame redundant? Zumindest steht bei addGroupToGame ja, dass es replaced wird, wenn's existiert

sneeex suggested changes for liquid-development/game-tracker#67 2025-11-26 12:40:47 +00:00
CreateGameView erstellen

Und ruleset_list_tile bitte umbenennen zu was generellerem, nicht auf ruleset bezogen

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-26 12:39:45 +00:00
CreateGameView erstellen

Hä aber das sind doch rulesets und nicht gametypes?? Ist ja das file choose_ruleset_view

sneeex commented on pull request liquid-development/game-tracker#62 2025-11-26 12:08:56 +00:00
GameResultView erstellen

Review macht noch keinen Sinn, ich warte auf die entsprechenden PRs

Aber kannst dir doch das schon mal angucken, das wird ja sonst

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-26 12:05:47 +00:00
CreateGameView erstellen

Da steht noch gametypes obwohls ruleset ist

9ba3dd7909 added missing consts
sneeex opened issue liquid-development/game-tracker#75 2025-11-25 22:20:43 +00:00
Group erstellen nur möglich mit min. 2 spielern
sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

Macht keinen Sinn, da gametype und rulesets ausgewählt werden können

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

warum so viele comments? ist doch klar wofür die settings jeweils sind, also label indicator, divider etc

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

auch hier no gametypes und nicht irgendwas anderes bzw gametypes or rulesets

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

warum auch hier bei allen vars comments??

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

mehrere prints noch im code

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

auch hier print

sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen
grafik.png
sneeex commented on pull request liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen

0.9, 0.95 sieht komisch aus mit der navbar, soll so sein wie im groupsview

sneeex suggested changes for liquid-development/game-tracker#67 2025-11-25 22:19:34 +00:00
CreateGameView erstellen