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 +8 -2
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d8abad6fd8 - Show all commits

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

@@ -1,9 +1,15 @@
import 'package:drift/drift.dart';
import 'package:game_tracker/data/db/tables/game_table.dart';
import 'package:game_tracker/data/db/tables/group_table.dart';
class MatchTable extends Table {
TextColumn get id => text()();
TextColumn get name => text()();
late final winnerId = text().nullable()();
TextColumn get name => text().nullable()();
TextColumn get gameId =>
text().references(GameTable, #id, onDelete: KeyAction.cascade)();
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
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 notes => 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
DateTimeColumn get createdAt => dateTime()();
@override