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

StatisticsView erstellen #30

Merged
sneeex merged 6 commits from feature/6-statisticsview-erstellen into development 2025-11-23 12:59:40 +00:00
Conversation 3 Commits 6 Files Changed 2 +9 -3
flixcoo commented 2025-11-17 18:29:40 +00:00
Owner
Copy Link

StatisticsView erstellen

Zugehörige Issue(s):
Closes #6

Beschreibung

Es wurde eine Statistik-Ansicht mit beispielhaften Kacheln hinzugefügt. Verschiedene Statistiken wären mit komplexeren Daten möglich.

Änderungen

  • Konfigurierbares StatisticsTile implementiert
  • Drei verschiedene Statistiken implementiert:
    • Gespielte Spiele pro Spieler
    • Gewonnene Spiele pro Spieler
    • Winrate pro Spieler
# StatisticsView erstellen **Zugehörige Issue(s):** Closes #6 ## Beschreibung Es wurde eine Statistik-Ansicht mit beispielhaften Kacheln hinzugefügt. Verschiedene Statistiken wären mit komplexeren Daten möglich. ## Änderungen - Konfigurierbares `StatisticsTile` implementiert - Drei verschiedene Statistiken implementiert: - Gespielte Spiele pro Spieler - Gewonnene Spiele pro Spieler - Winrate pro Spieler
flixcoo added this to the Minimum Viable Product (MVP) milestone 2025-11-17 18:29:40 +00:00
flixcoo self-assigned this 2025-11-17 18:29:40 +00:00
flixcoo added 2 commits 2025-11-22 21:10:24 +00:00
Implemented StatisticsWidget tile 310b9aa43b
Implemented first version of statistics view
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m5s
Details
b2036e4e68
flixcoo added 4 commits 2025-11-22 22:30:27 +00:00
Changed values attribute & maxBarWidth 59c041699d
Added new metric & changed layout builder of Skeletonizer e60961730f
changed skeletonizer transition duration back to normal fba35521cb
Docs, small changes
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m13s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m15s
Details
feb5fa0615
flixcoo added the
PR
Ready for Review
2
 label 2025-11-22 22:30:37 +00:00
flixcoo requested review from gelbeinhalb 2025-11-22 22:30:42 +00:00
flixcoo requested review from sneeex 2025-11-22 22:30:43 +00:00
flixcoo requested review from Owners 2025-11-22 22:30:45 +00:00
flixcoo removed review request for gelbeinhalb 2025-11-22 22:30:48 +00:00
flixcoo removed review request for sneeex 2025-11-22 22:30:49 +00:00
flixcoo changed title from WIP: StatisticsView erstellen to StatisticsView erstellen 2025-11-22 22:30:53 +00:00
flixcoo added 1 commit 2025-11-22 23:36:40 +00:00
Merge branch 'development' into feature/6-statisticsview-erstellen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m33s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m33s
Details
acc5b0a3e9
sneeex requested changes 2025-11-23 11:09:42 +00:00
Dismissed
lib/presentation/views/main_menu/statistics_view.dart Outdated
@@ -9,0 +89,4 @@
),
SizedBox(height: constraints.maxHeight * 0.02),
StatisticsTile(
icon: Icons.casino,
sneeex commented 2025-11-23 10:54:49 +00:00
Owner
Copy Link

hier vielleicht besser irgendein Pokal icon oder so? Weil so ist das doppelt, das blöd

hier vielleicht besser irgendein Pokal icon oder so? Weil so ist das doppelt, das blöd
flixcoo commented 2025-11-23 11:15:40 +00:00
Author
Owner
Copy Link

fixed

fixed
sneeex marked this conversation as resolved
lib/presentation/views/main_menu/statistics_view.dart Outdated
@@ -9,0 +129,4 @@
final winner = game.winner;
if (winner != null && winner.isNotEmpty) {
final index = winCounts.indexWhere((entry) => entry.$1 == winner);
if (index != -1) {
sneeex commented 2025-11-23 10:58:01 +00:00
Owner
Copy Link

maybe comment, dass -1 heißt nicht gefunden, war mir jetzt nicht klar.
Falls du's machst, natürlich bei allen

maybe comment, dass -1 heißt nicht gefunden, war mir jetzt nicht klar. Falls du's machst, natürlich bei allen
flixcoo commented 2025-11-23 11:13:20 +00:00
Author
Owner
Copy Link

Habs jetzt mal für euch hinzugefügt, weil ihr ja beide relativ neu seid, ist halt aber einfach der return Wert dieser Funktion. Können es dann bei Zeiten wieder entfernen

Habs jetzt mal für euch hinzugefügt, weil ihr ja beide relativ neu seid, ist halt aber einfach der return Wert dieser Funktion. Können es dann bei Zeiten wieder entfernen
👍 1
sneeex marked this conversation as resolved
lib/presentation/views/main_menu/statistics_view.dart
@@ -9,0 +242,4 @@
: 0;
return (name, percent);
}).toList();
sneeex commented 2025-11-23 11:07:12 +00:00
Owner
Copy Link

Aktuell wird mir bei einem Spieler als Winrate 0.0 angezeigt, das sollte entweder 0 sein oder 0.00, aber nicht 0.0.
Meine Präferenz ist glaube ich 0.00, weil dann alles in einer Line ist.
Daten im Anhang.

Aktuell wird mir bei einem Spieler als Winrate 0.0 angezeigt, das sollte entweder 0 sein oder 0.00, aber nicht 0.0. Meine Präferenz ist glaube ich 0.00, weil dann alles in einer Line ist. Daten im Anhang.
game_tracker-data.json
9.2 KiB
flixcoo commented 2025-11-23 11:17:34 +00:00
Author
Owner
Copy Link

Ja das hat mich auch genervt, ich muss mal schauen ob das irgendwie geht, aber ich glaube nicht. Das liegt halt an double, bei dem nur die Zahlen über 0 angezeigt werden (also z.B. 0.10 wird zu 0.1). Ggf. könnte man den Datentyp auf String ändern, das würde die Berechnungen anstrengend machen

Ja das hat mich auch genervt, ich muss mal schauen ob das irgendwie geht, aber ich glaube nicht. Das liegt halt an `double`, bei dem nur die Zahlen über 0 angezeigt werden (also z.B. 0.10 wird zu 0.1). Ggf. könnte man den Datentyp auf String ändern, das würde die Berechnungen anstrengend machen
sneeex commented 2025-11-23 11:21:51 +00:00
Owner
Copy Link

Ja das hat mich auch genervt, ich muss mal schauen ob das irgendwie geht, aber ich glaube nicht. Das liegt halt an double, bei dem nur die Zahlen über 0 angezeigt werden (also z.B. 0.10 wird zu 0.1). Ggf. könnte man den Datentyp auf String ändern, das würde die Berechnungen anstrengend machen

such halt oder mach shitty string, wo du einfach guckst ob zwei stellen nach dem Punkt sind

> Ja das hat mich auch genervt, ich muss mal schauen ob das irgendwie geht, aber ich glaube nicht. Das liegt halt an `double`, bei dem nur die Zahlen über 0 angezeigt werden (also z.B. 0.10 wird zu 0.1). Ggf. könnte man den Datentyp auf String ändern, das würde die Berechnungen anstrengend machen such halt oder mach shitty string, wo du einfach guckst ob zwei stellen nach dem Punkt sind
flixcoo commented 2025-11-23 11:33:29 +00:00
Author
Owner
Copy Link

done, doch einfacher als gedacht

done, doch einfacher als gedacht
sneeex commented 2025-11-23 11:17:14 +00:00
Owner
Copy Link

und einmal dev noch reinmergen

und einmal dev noch reinmergen
👍 1
flixcoo added 2 commits 2025-11-23 11:18:09 +00:00
Added comments for return value -1 fee5c57207
Changed icon for second statistics tile
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m21s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m22s
Details
d411f58134
flixcoo added 1 commit 2025-11-23 11:19:22 +00:00
Merge branch 'development' into feature/6-statisticsview-erstellen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m25s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m25s
Details
c38c731b41 
# Conflicts:
#	lib/presentation/views/main_menu/statistics_view.dart
flixcoo added 1 commit 2025-11-23 11:33:18 +00:00
Changed double depiction
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m20s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m23s
Details
e9b041e43a
flixcoo requested review from sneeex 2025-11-23 11:33:33 +00:00
flixcoo added 1 commit 2025-11-23 11:34:47 +00:00
Changed item count back to normal
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m19s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m23s
Details
7cda25a380
sneeex approved these changes 2025-11-23 12:59:27 +00:00
sneeex left a comment
Owner
Copy Link

geile nummer

geile nummer
sneeex merged commit 17e882986d into development 2025-11-23 12:59:40 +00:00
sneeex deleted branch feature/6-statisticsview-erstellen 2025-11-23 12:59:40 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
sneeex
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 flixcoo
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#30
Delete Branch "feature/6-statisticsview-erstellen"

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?