Felix Kirchner flixcoo
1 Followers · 1 Following
Repositories Projects Packages Public Activity Starred Repositories
flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 20:11:06 +00:00
Bearbeiten und Löschen von Gruppen

Eigentlich nicht, das ist schon wieder Fusch am Bau

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 20:05:54 +00:00
Bearbeiten und Löschen von Gruppen

ja schlimm

flixcoo suggested changes for liquid-development/game-tracker#148 2026-03-09 20:04:08 +00:00
Bearbeiten und Löschen von Gruppen
flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 20:04:08 +00:00
Bearbeiten und Löschen von Gruppen

Du sollst direkt die Datenbankanfrage so gestalten, dass du nur die Matches bekommst, welche die Gruppe enthalten. Das war mein Punkt

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:44:55 +00:00
Bearbeiten und Löschen von Gruppen

ja aber da diese vorgehensweise jetzt mehr als einmal im code ist machts erst recht sinn dafür ne methode zu schreiben und entsprechende vorkommen zu ersetzen

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:36:58 +00:00
Bearbeiten und Löschen von Gruppen

wo hab ich das gemacht? File und Zeile

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:32:55 +00:00
Bearbeiten und Löschen von Gruppen

Setzt es mal in der Methode anfangs auf true, so ists z.B. auch im GroupView

flixcoo suggested changes for liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Review

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Warum wird isLoading nicht in _loadStatistics auf true gesetzt?

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Lieber Methode getMatchesToGroup() in groupDao.dart, kann ggf. später auch noch einmal verwendet werden

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Kurzen doc comment nochmal zu dieser Methode vllt

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Kommentar auf Deutsch

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Methode lieber sowas nennen removeGroupFromMatch da hier ja keine keine Gruppe gelöscht wird

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Unnötiger Kommentar

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Unnötiger Kommentar & auf deutsch

flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-09 19:30:28 +00:00
Bearbeiten und Löschen von Gruppen

Unnötiger Kommentar & auf deutsch

9b2fcf1860 Fix: Linter exclude
flixcoo commented on pull request liquid-development/game-tracker#194 2026-03-08 21:26:32 +00:00
WIP: Verschiedene Regelsätze implementieren

Wartet auf #191

f0c575d2c9 Implemented different result view depending on ruleset
d5a7bb320f Implememented different result tiles in match detail view for different rulesets
Compare 2 commits »
flixcoo commented on pull request liquid-development/game-tracker#148 2026-03-08 18:46:24 +00:00
Bearbeiten und Löschen von Gruppen

Ich habe ein game, das ist fertig und da ist jemand Winner. Player: 1, 2, 3, 4, Gruppenspieler: 1, 2, 3 Winner Spieler: 1

Jetzt bearbeite ich die Gruppe und entferne Player 1 …