Felix Kirchner flixcoo
flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 21:35:06 +00:00
Neue Datenbank Struktur

Finde null sollte man nur dann verwenden, wenns halt nicht anders geht, wie z.B. bei objekten

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 21:26:36 +00:00
Neue Datenbank Struktur

Äh ne ich glaub das ist irrellevant, das hatte ich vor 5 Tagen schonmal kommentiert aber das review nicht abgeschickt

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 20:39:46 +00:00
Neue Datenbank Struktur

Aber das funktioniert doch trotzdem. Du gibst je nach Spielmodus und Score in diesem Spielmodus ein Player Objekt zurück. Da wir aktuell sowieso nur Spiele mit winner only haben, gibst du…

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 20:37:24 +00:00
Neue Datenbank Struktur

Das Problem ist, wenn du sagst Pokerkarten als Icon, musst du ja total viele Icons irgendwo her bekommen, die sehr speziell sind (Spielkarten, Brettspiel, etc ..), da würde ich dann vllt eher auf…

flixcoo suggested changes for liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Review

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Hier bitte ergänzen das alle Parameter (id, name, ...) auch den ursprünglichen parametern entsprechen

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Leerer String statt nullable

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Nicht nullable sondern empty string

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Gleiches wie oben, nicht nullable machen sondern einfach als Standard n leeren String

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Gleiche anmerkung wie oben, würde ich über einen int lösen

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Ich weiß nicht, was wir besprochen hatten, aber macht nicht mehr sinn

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Spielerliste auch nicht optional

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Game nicht optional sondern required

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Hier auch nicht nullable sondern empty string

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Hier auch nicht nullable, sondern empty string

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Name ist sollte nicht nullable sein, der wird im Frontend doch gesetzt wenn man keinen eigenen eingibt

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Kommentar drüber setzten, damit die Zeile nicht so lang ist

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Hier auch wieder nicht nullable machen sondern als leeren String setzen

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Meinst du hier matchId?

flixcoo commented on pull request liquid-development/game-tracker#156 2026-01-21 17:08:49 +00:00
Neue Datenbank Struktur

Warum hier nicht