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

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

@@ -4,11 +4,11 @@ import 'package:game_tracker/data/db/tables/group_table.dart';
class MatchTable extends Table { class MatchTable extends Table {
TextColumn get id => text()(); TextColumn get id => text()();
TextColumn get name => text().nullable()();
TextColumn get gameId => TextColumn get gameId =>
text().references(GameTable, #id, onDelete: KeyAction.cascade)(); text().references(GameTable, #id, onDelete: KeyAction.cascade)();
TextColumn get groupId => TextColumn get groupId =>
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.ä.
text().references(GroupTable, #id, onDelete: KeyAction.cascade).nullable()(); // Nullable if not part of a group text().references(GroupTable, #id, onDelete: KeyAction.cascade).nullable()(); // Nullable if not part of a group
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
TextColumn get name => text().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 notes => text().nullable()(); 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()(); DateTimeColumn get createdAt => dateTime()();