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

FutureBuilder-/Skeleton-/Futurelogik verbessern #90

Merged
sneeex merged 24 commits from enhancement/80-FutureBuilder-Logik-bei-Interaktion-mit-DB-verbessern into development 2025-12-29 17:21:49 +00:00
Conversation 11 Commits 24 Files Changed 7 +261 -352
sneeex commented 2025-12-21 18:55:04 +00:00
Owner
Copy Link

FutureBuilder-/Skeleton-/Futurelogik verbessern

Zugehörige Issue(s):
Closes #80

Beschreibung

Futurebuilder aus allen Widgets entfernt und die minimumSkeletonDuration überall hinzugefügt.

Änderungen

  • in allen Views die FutureBuilder genutzt haben diesen entfernt
  • in allen Views mit Skeleton Loading die minimumSkeletonDuration hinzugefügt
    • bei gehen von CreateGroupView auf GroupView und CreateMatchView/GameMatchView auf MatchView ist die minimumSkeletonDuration absichtlich nicht drin
  • teilweise DB und Future logik überarbeitet
# FutureBuilder-/Skeleton-/Futurelogik verbessern **Zugehörige Issue(s):** Closes #80 ## Beschreibung Futurebuilder aus allen Widgets entfernt und die `minimumSkeletonDuration` überall hinzugefügt. ## Änderungen - [x] in allen Views die FutureBuilder genutzt haben diesen entfernt - [x] in allen Views mit Skeleton Loading die `minimumSkeletonDuration` hinzugefügt - [x] bei gehen von CreateGroupView auf GroupView und CreateMatchView/GameMatchView auf MatchView ist die `minimumSkeletonDuration` absichtlich nicht drin - [x] teilweise DB und Future logik überarbeitet
sneeex added this to the Minimum Viable Product (MVP) milestone 2025-12-21 18:55:04 +00:00
sneeex added the
Workflow
In Progress
3
 label 2025-12-21 18:55:04 +00:00
sneeex self-assigned this 2025-12-21 18:55:04 +00:00
sneeex added 7 commits 2025-12-21 18:55:04 +00:00
added constant minimumSkeletonDuration 0eaf3d251b
Changed futurebuilder logic in groups_view d96494f608
removed futurebuilder logic in groups_view f05114a99e
changed futurebuilder logic in player selection 76186787e7
changed skeleton duration logic in statistics view and refactored c8532adfde
added minimumSkeletonDuration constant and changed future logic 24b60bb18b
refactor
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Failing after 2m6s
Details
4f0a1eec6d
sneeex changed title from FutureBuilder-/Skeleton-/Futurelogik verbessern to WIP: FutureBuilder-/Skeleton-/Futurelogik verbessern 2025-12-21 19:01:27 +00:00
sneeex added 1 commit 2025-12-21 19:01:52 +00:00
add const
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m22s
Details
df0a5207c2
sneeex commented 2025-12-21 19:04:45 +00:00
Author
Owner
Copy Link

fragen:

  • alles so umbauen, dass es kein futurebuilder mehr nutzt? aktuell wird da wo nur auf eine abfrage gewartet wird noch mit futurebuilder gearbeitet
  • minimumSkeletonDuration auch nutzen, wenn man von CreateGroupView/CreateMatchView zurückgeht?
fragen: - alles so umbauen, dass es kein futurebuilder mehr nutzt? aktuell wird da wo nur auf eine abfrage gewartet wird noch mit futurebuilder gearbeitet - `minimumSkeletonDuration` auch nutzen, wenn man von CreateGroupView/CreateMatchView zurückgeht?
sneeex added 5 commits 2025-12-23 22:17:33 +00:00
remove futurebuilder from playerselection and refactor 9b3d61e5b0
remove futurebuilder from groups view and refactor 9ad5c4ad6f
remove futurebuilder from player selection and refactor 06a9c0cd84
refactor group loading into reusable method loadGroups and call it after adding a group a747d91c5d
remove futurebuilder from match view and refactor
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m10s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m10s
Details
7732c6ceb9
sneeex added
PR
Ready for Review
2
 and removed
Workflow
In Progress
3
 labels 2025-12-23 22:23:38 +00:00
sneeex requested review from Owners 2025-12-23 22:24:28 +00:00
gelbeinhalb commented 2025-12-24 09:35:02 +00:00
Owner
Copy Link

Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo

Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo
sneeex commented 2025-12-24 09:36:18 +00:00
Author
Owner
Copy Link

Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo

Kann gerade nicht checken, aber lösch mal die App auf dem Simulator und starte neu und guck das du in der richtigen branch auf dem aktuellen Stand bist.

> Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo Kann gerade nicht checken, aber lösch mal die App auf dem Simulator und starte neu und guck das du in der richtigen branch auf dem aktuellen Stand bist.
gelbeinhalb requested changes 2025-12-24 09:39:47 +00:00
Dismissed
gelbeinhalb left a comment
Owner
Copy Link

Idk ob das mit den Punkten zu dieser PR gehören sollte, ist mir nur gerade aufgefallen.
Ansonsten sieht gut aus :)

Idk ob das mit den Punkten zu dieser PR gehören sollte, ist mir nur gerade aufgefallen. Ansonsten sieht gut aus :)
lib/presentation/views/main_menu/home_view.dart Outdated
@@ -193,0 +106,4 @@
visible: !isLoading,
replacement: const Center(
heightFactor: 12,
child: Text('No recent games available.'),
gelbeinhalb commented 2025-12-24 09:35:47 +00:00
Owner
Copy Link

Punkt entfernen

Punkt entfernen
sneeex marked this conversation as resolved
lib/presentation/views/main_menu/home_view.dart
@@ -193,0 +139,4 @@
] else ...[
const Center(
heightFactor: 4,
child: Text('No second game available.'),
gelbeinhalb commented 2025-12-24 09:35:59 +00:00
Owner
Copy Link

Punkt entfernen

Punkt entfernen
sneeex marked this conversation as resolved
lib/presentation/widgets/player_selection.dart
@@ -176,0 +177,4 @@
icon: Icons.info,
title: 'Info',
message: allPlayers.isEmpty
? 'No players created yet.'
gelbeinhalb commented 2025-12-24 09:37:46 +00:00
Owner
Copy Link

Punkt entfernen

Punkt entfernen
sneeex marked this conversation as resolved
lib/presentation/widgets/player_selection.dart Outdated
@@ -176,0 +179,4 @@
message: allPlayers.isEmpty
? 'No players created yet.'
: (selectedPlayers.length == allPlayers.length)
? 'No more players to add.'
gelbeinhalb commented 2025-12-24 09:37:52 +00:00
Owner
Copy Link

Punkt entfernen

Punkt entfernen
sneeex marked this conversation as resolved
lib/presentation/widgets/player_selection.dart
@@ -176,0 +180,4 @@
? 'No players created yet.'
: (selectedPlayers.length == allPlayers.length)
? 'No more players to add.'
: 'No players found with that name.',
gelbeinhalb commented 2025-12-24 09:37:59 +00:00
Owner
Copy Link

Punkt entfernen

Punkt entfernen
sneeex marked this conversation as resolved
gelbeinhalb commented 2025-12-24 09:40:43 +00:00
Owner
Copy Link

Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo

Kann gerade nicht checken, aber lösch mal die App auf dem Simulator und starte neu und guck das du in der richtigen branch auf dem aktuellen Stand bist.

Ja hab gerade eben erst deine Branch gepulled und war vorher in development. Ist alles auf dem neusten stand.
Ich guck mal mit dem löschen.

> > Bei mir sind alle Views im loading stuck bis auf das Groups View. Ist das ein Problem bei mir oder ist das bei euch auch so? @sneeex @flixcoo > > Kann gerade nicht checken, aber lösch mal die App auf dem Simulator und starte neu und guck das du in der richtigen branch auf dem aktuellen Stand bist. Ja hab gerade eben erst deine Branch gepulled und war vorher in development. Ist alles auf dem neusten stand. Ich guck mal mit dem löschen.
gelbeinhalb commented 2025-12-24 09:43:34 +00:00
Owner
Copy Link

image.png

Alles andere Klappt jetzt

![image.png](/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c) Alles andere Klappt jetzt
image.png
55 KiB
sneeex commented 2025-12-24 11:06:35 +00:00
Author
Owner
Copy Link

image.png

Alles andere Klappt jetzt

kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall?

> ![image.png](/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c) > > Alles andere Klappt jetzt kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall?
sneeex added 1 commit 2025-12-24 11:53:40 +00:00
fix rangeerror when only 1 or less matches exist
Some checks failed
Pull Request Pipeline / test (pull_request) Successful in 2m21s
Details
Pull Request Pipeline / lint (pull_request) Failing after 2m34s
Details
1d92084da6
sneeex added 1 commit 2025-12-24 11:54:09 +00:00
remove double quotes
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m15s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m21s
Details
b29cd6dff4
sneeex added 4 commits 2025-12-24 12:01:42 +00:00
removed empty line f2917a6813
wrap isLoading = false in a mounted check and setState call f9722bc762
add mounted check before calling setState in match_view 7eb25221d7
wrap isLoading into mounted and setState
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m8s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
Details
1e730cebe6
sneeex added 1 commit 2025-12-24 12:05:53 +00:00
put isLoading in existing setState and move mounted check up
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
Details
c73f37507f
sneeex added 1 commit 2025-12-24 12:08:28 +00:00
adjust heightFactor for empty game state in home view
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
Details
7a80c1a792
sneeex changed title from WIP: FutureBuilder-/Skeleton-/Futurelogik verbessern to FutureBuilder-/Skeleton-/Futurelogik verbessern 2025-12-24 12:42:11 +00:00
sneeex requested review from gelbeinhalb 2025-12-24 12:42:26 +00:00
sneeex added 2 commits 2025-12-24 21:53:21 +00:00
move setState to include match loading and sorting in MatchView b0a5145490
remove dots after sentences
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m4s
Details
12856342a8
gelbeinhalb commented 2025-12-25 01:20:11 +00:00
Owner
Copy Link

!image.png(/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c)

Alles andere Klappt jetzt

kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall?

Ja stimmt bei mir auch nur. Ich meinte ja heute Mittag, dass das auch auftritt, wenn man Daten importiert hat, was wohl daran liegt dass mein Data import anscheinend keine Matches enthält oder broken ist.

> > !image.png(/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c) > > > > Alles andere Klappt jetzt > > kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall? Ja stimmt bei mir auch nur. Ich meinte ja heute Mittag, dass das auch auftritt, wenn man Daten importiert hat, was wohl daran liegt dass mein Data import anscheinend keine Matches enthält oder broken ist.
sneeex commented 2025-12-25 09:55:15 +00:00
Author
Owner
Copy Link

!image.png(/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c)

Alles andere Klappt jetzt

kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall?

Ja stimmt bei mir auch nur. Ich meinte ja heute Mittag, dass das auch auftritt, wenn man Daten importiert hat, was wohl daran liegt dass mein Data import anscheinend keine Matches enthält oder broken ist.

ja genau import geht grad nicht, also dann kann das ja reviewt werden nochmal, oder?

> > > !image.png(/attachments/234ead1e-eac7-4f64-91b9-b81cccce608c) > > > > > > Alles andere Klappt jetzt > > > > kann das nur recreaten wenn keine games vorhanden sind, ist das auch nur dann der fall? > > Ja stimmt bei mir auch nur. Ich meinte ja heute Mittag, dass das auch auftritt, wenn man Daten importiert hat, was wohl daran liegt dass mein Data import anscheinend keine Matches enthält oder broken ist. ja genau import geht grad nicht, also dann kann das ja reviewt werden nochmal, oder?
sneeex requested review from flixcoo 2025-12-25 09:55:22 +00:00
gelbeinhalb approved these changes 2025-12-26 09:12:22 +00:00
gelbeinhalb left a comment
Owner
Copy Link

Sieht gut aus!

Sieht gut aus!
flixcoo approved these changes 2025-12-29 17:15:03 +00:00
sneeex added 1 commit 2025-12-29 17:19:30 +00:00
Merge remote-tracking branch 'origin/development' into enhancement/80-FutureBuilder-Logik-bei-Interaktion-mit-DB-verbessern
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m13s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m17s
Details
88bea48956 
# Conflicts:
#	lib/presentation/views/main_menu/match_view/match_view.dart
sneeex scheduled this pull request to auto merge when all checks succeed 2025-12-29 17:19:51 +00:00
sneeex merged commit 94fdd7026d into development 2025-12-29 17:21:49 +00:00
sneeex deleted branch enhancement/80-FutureBuilder-Logik-bei-Interaktion-mit-DB-verbessern 2025-12-29 17:21:49 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
gelbeinhalb
flixcoo
Labels
Clear labels
Duplicate
This issue or pull request already exists
Layer
BE
Issues/PRs related to the backend
Layer
DB
Issues/PRs related to the database
Layer
FE
Issues/PRs related to the frontend
Layer
Full Stack
Issues/PRs spanning multiple layers (frontend, backend, database)
PR
In Review
1
Pull Request that is reviewed
PR
Ready for Review
2
Pull Requests which are ready to be reviewed
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Task
Bug
Reports of unexpected behavior or defects in existing functionality.
Task
Documentation
Tasks related to improving or adding documentation (READMEs, code comments, guides).
Task
Enhancement
Improvements or optimizations to existing features (non-breaking changes).
Task
Feature
Requests for new functionality or significant additions to the project.
Task
Refactoring
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
Workflow
Abandoned
4
Somebody has started to work on this but abandoned work
Workflow
Backlog
4
Issues/PRs that are acknowledged but not being actively worked on yet.
Workflow
In Progress
3
Issues/PRs currently being actively worked on.
Workflow
On Hold
3
Issues/PRs temporarily being paused or blocked.
Workflow
Won't Fix
5
This issue won't be fixed
No Label
PR
Ready for Review
2
Milestone
No items
No Milestone Minimum Viable Product (MVP)
Assignees
Clear assignees
flixcoo gelbeinhalb sneeex
No Assignees sneeex
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#90
Delete Branch "enhancement/80-FutureBuilder-Logik-bei-Interaktion-mit-DB-verbessern"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?