liquid-development/game-tracker
3
1
Fork 0
Code Issues 8 Pull Requests 4 Actions Releases 1 Activity

Neue Datenbank Struktur #156

Open
gelbeinhalb wants to merge 88 commits from feature/88-neue-datenbank-struktur into development
pull from: feature/88-neue-datenbank-struktur
merge into: liquid-development:development
Conversation 15 Commits 88 Files Changed 61 +2 -1
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ccfea71a35 - Show all commits

3
lib/data/db/tables/match_table.dart
View File

@@ -6,8 +6,9 @@ class MatchTable extends Table {
TextColumn get id => text()();
TextColumn get gameId =>
text().references(GameTable, #id, onDelete: KeyAction.cascade)();
// Nullable if not part of a group
gelbeinhalb marked this conversation as resolved
flixcoo commented 2026-02-23 17:12:24 +00:00
Review
Copy Link

Kommentar mach keinen Sinn, sollte eher heißen Nullable if not group takes part in the match o.ä.

Kommentar mach keinen Sinn, sollte eher heißen `Nullable if not group takes part in the match` o.ä.
TextColumn get groupId =>
gelbeinhalb marked this conversation as resolved
flixcoo commented 2026-01-21 16:46:53 +00:00
Review
Copy Link

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

Kommentar drüber setzten, damit die Zeile nicht so lang ist
👍 1
text().references(GroupTable, #id, onDelete: KeyAction.cascade).nullable()(); // Nullable if not part of a group
text().references(GroupTable, #id, onDelete: KeyAction.cascade).nullable()();
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2026-01-21 16:47:34 +00:00
Outdated
Review
Copy Link

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

Name ist sollte nicht nullable sein, der wird im Frontend doch gesetzt wenn man keinen eigenen eingibt
👍 1
TextColumn get name => text().nullable()();
gelbeinhalb marked this conversation as resolved Outdated
flixcoo commented 2026-01-21 16:47:51 +00:00
Outdated
Review
Copy Link

Hier auch nicht nullable, sondern empty string

Hier auch nicht nullable, sondern empty string
👍 1
TextColumn get notes => text().nullable()();
DateTimeColumn get createdAt => dateTime()();