MVP-Refactoring #139

Merged
flixcoo merged 20 commits from refactoring/68-mvp-refactoring into development 2026-01-08 20:24:01 +00:00
Owner

MVP-Refactoring

Zugehörige Issue(s):
Closes #68

Beschreibung

Großes Refactoring, inkl Ergänzung von Kommentaren, Optimisierungen von den build()-Methoden, Anpassungen von Übersetzungen

Änderungen

  • Kommentare für alle relevanten Klassen, Funktionen und Variablen hinzugefügt
  • Lokalisierungen teilweise angepasst
  • Alle build()-Methoden auf Optimierung überprüft
  • CustomTheme erweitert und Änderungen in die App propagiert

Zusätzliche Anmerkungen

Die MatchSummaryTile wurde wegen #137 nicht refactored

# MVP-Refactoring **Zugehörige Issue(s):** Closes #68 ## Beschreibung Großes Refactoring, inkl Ergänzung von Kommentaren, Optimisierungen von den `build()`-Methoden, Anpassungen von Übersetzungen ## Änderungen - [x] Kommentare für alle relevanten Klassen, Funktionen und Variablen hinzugefügt - [x] Lokalisierungen teilweise angepasst - [x] Alle `build()`-Methoden auf Optimierung überprüft - [x] `CustomTheme` erweitert und Änderungen in die App propagiert ## Zusätzliche Anmerkungen Die `MatchSummaryTile` wurde wegen #137 nicht refactored
flixcoo added this to the Minimum Viable Product (MVP) milestone 2026-01-07 12:42:35 +00:00
flixcoo self-assigned this 2026-01-07 12:42:35 +00:00
flixcoo added 13 commits 2026-01-07 13:55:00 +00:00
flixcoo added 2 commits 2026-01-07 16:02:15 +00:00
# Conflicts:
#	lib/l10n/generated/app_localizations.dart
#	lib/l10n/generated/app_localizations_de.dart
#	lib/l10n/generated/app_localizations_en.dart
Updated localizations
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 2m11s
Pull Request Pipeline / test (pull_request) Successful in 2m10s
d5ee6449b0
flixcoo added 1 commit 2026-01-07 16:30:28 +00:00
Updated localizations
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Pull Request Pipeline / lint (pull_request) Successful in 2m18s
76121eb4fb
flixcoo requested review from gelbeinhalb 2026-01-07 16:44:11 +00:00
flixcoo requested review from sneeex 2026-01-07 16:44:11 +00:00
flixcoo added the
PR
Ready for Review
2
label 2026-01-07 16:44:15 +00:00
flixcoo changed title from WIP: MVP-Refactoring to MVP-Refactoring 2026-01-07 16:44:17 +00:00
sneeex requested changes 2026-01-07 17:20:24 +00:00
Dismissed
@@ -19,4 +19,3 @@
class _CreateGroupViewState extends State<CreateGroupView> {
final _groupNameController = TextEditingController();
late final AppDatabase db;
Owner

warum für den rest comments aber nicht für db?

warum für den rest comments aber nicht für db?
Author
Owner

Habe Datenbank und isLoading nicht kommentiert, weil die so allgegenwärtig sind, dass die eigentlich keinen kommentar brauchen. Soll ich einen hinzufügen?

Habe Datenbank und `isLoading` nicht kommentiert, weil die so allgegenwärtig sind, dass die eigentlich keinen kommentar brauchen. Soll ich einen hinzufügen?
sneeex marked this conversation as resolved
@@ -53,3 +49,1 @@
),
centerTitle: true,
),
appBar: AppBar(title: Text(loc.create_new_group)),
Owner

schatz irgendwas passt hier nicht:
grafik.png

warum hast du alles weggelassen außer den title??? das ganze design ist anders jetzt und kacke auch

schatz irgendwas passt hier nicht: ![grafik.png](/attachments/2e570ad8-7b7b-4551-bb0c-576a4704f5f2) warum hast du alles weggelassen außer den title??? das ganze design ist anders jetzt und kacke auch
sneeex marked this conversation as resolved
@@ -21,7 +21,11 @@ class GroupsView extends StatefulWidget {
class _GroupsViewState extends State<GroupsView> {
late final AppDatabase db;
Owner

hier ebenfalls kein comment?

hier ebenfalls kein comment?
flixcoo marked this conversation as resolved
@@ -21,9 +21,17 @@ class HomeView extends StatefulWidget {
class _HomeViewState extends State<HomeView> {
bool isLoading = true;
Owner

hier ebenfalls kein comment?

hier ebenfalls kein comment?
flixcoo marked this conversation as resolved
@@ -38,4 +40,2 @@
appBar: AppBar(
backgroundColor: CustomTheme.backgroundColor,
scrolledUnderElevation: 0,
leading: IconButton(
Owner

auch hier mit dem design weird alles aufeinmal anders wie sonst
grafik.png

auch hier mit dem design weird alles aufeinmal anders wie sonst ![grafik.png](/attachments/93727713-90f7-42d7-ae20-1444d8709b44)
flixcoo marked this conversation as resolved
@@ -58,3 +53,2 @@
),
centerTitle: true,
title: Text(loc.choose_group),
),
Owner

grafik.png

![grafik.png](/attachments/a9c6941a-0805-49e8-8585-7c0a56c3ce4d)
flixcoo marked this conversation as resolved
@@ -138,3 +132,2 @@
/// Filters the groups based on the search query.
/// TODO: Maybe implement also targetting player names?
/// Filters the groups based on the search [query].
Owner

warum query in klammern? entweder ganz oder garnicht aber wat soll dat hier

warum query in klammern? entweder ganz oder garnicht aber wat soll dat hier
Author
Owner

Weil das der Input Parameter der Funktion ist

Weil das der Input Parameter der Funktion ist
sneeex marked this conversation as resolved
@@ -40,4 +40,3 @@
scrolledUnderElevation: 0,
leading: IconButton(
icon: const Icon(Icons.arrow_back_ios),
onPressed: () {
Owner

grafik.png

![grafik.png](/attachments/357e0556-4575-4f60-b812-dc7b582bf8b6)
flixcoo marked this conversation as resolved
@@ -59,3 +54,2 @@
),
centerTitle: true,
title: Text(widget.match.name),
),
Owner

grafik.png

![grafik.png](/attachments/06706384-fe63-4ea8-8886-07737018c7f5)
flixcoo marked this conversation as resolved
@@ -28,6 +28,8 @@ class _MatchViewState extends State<MatchView> {
late final AppDatabase db;
Owner

kein comment?

kein comment?
flixcoo marked this conversation as resolved
@@ -28,6 +28,8 @@ class _MatchViewState extends State<MatchView> {
late final AppDatabase db;
bool isLoading = true;
Owner

kein comment?

kein comment?
flixcoo marked this conversation as resolved
@@ -31,3 +43,1 @@
List<Player> selectedPlayers = [];
List<Player> suggestedPlayers = [];
List<Player> allPlayers = [];
late final AppDatabase db;
Owner

kein comment?

kein comment?
flixcoo marked this conversation as resolved
@@ -32,3 +43,2 @@
List<Player> suggestedPlayers = [];
List<Player> allPlayers = [];
late final AppDatabase db;
bool isLoading = true;
Owner

kein comment?

kein comment?
flixcoo marked this conversation as resolved
gelbeinhalb approved these changes 2026-01-08 14:09:37 +00:00
Dismissed
gelbeinhalb left a comment
Owner

Sieht gut aus :)
gucke später nochmal genauer, hatte bei der Arbeit leider keine Zeit und muss auch jetzt schon direkt wieder weg. Aber hab nichts offensichtlich falsches im code gefunden, außer den Sachen, die Mathis schon gesagt hat.

Könnt also mergen :) wenn ich was finde mach ich einfach nen issue

Sieht gut aus :) gucke später nochmal genauer, hatte bei der Arbeit leider keine Zeit und muss auch jetzt schon direkt wieder weg. Aber hab nichts offensichtlich falsches im code gefunden, außer den Sachen, die Mathis schon gesagt hat. Könnt also mergen :) wenn ich was finde mach ich einfach nen issue
flixcoo added 1 commit 2026-01-08 20:07:36 +00:00
Fixed navbar issue
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
d741990f2f
flixcoo dismissed gelbeinhalb's review 2026-01-08 20:07:37 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

flixcoo added 1 commit 2026-01-08 20:13:28 +00:00
Updated localizations
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m14s
Pull Request Pipeline / lint (pull_request) Successful in 2m18s
cfb07bfe28
flixcoo added 1 commit 2026-01-08 20:15:05 +00:00
Remove Whitespace
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m12s
Pull Request Pipeline / lint (pull_request) Successful in 2m15s
2a1573ee2d
sneeex approved these changes 2026-01-08 20:19:38 +00:00
Dismissed
flixcoo added 1 commit 2026-01-08 20:21:38 +00:00
Small loc corrections
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Pull Request Pipeline / lint (pull_request) Successful in 2m21s
88766652b9
flixcoo dismissed sneeex's review 2026-01-08 20:21:38 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

sneeex approved these changes 2026-01-08 20:22:57 +00:00
flixcoo scheduled this pull request to auto merge when all checks succeed 2026-01-08 20:23:33 +00:00
flixcoo merged commit c2394c3733 into development 2026-01-08 20:24:01 +00:00
flixcoo deleted branch refactoring/68-mvp-refactoring 2026-01-08 20:24:02 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#139