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

Kennzeichnung für doppelte Spielernamen #200

Merged
flixcoo merged 9 commits from enhancement/182-kennzeichnung-fuer-doppelte-spielernamen into development 2026-04-24 09:18:44 +00:00
Conversation 6 Commits 9 Files Changed 15 +659 -176
flixcoo commented 2026-04-19 21:34:54 +00:00
Owner
Copy Link

Zugehörige Issue(s)

Closes #182

Beschreibung

Im Interface wird jetzt ein kleines #<num> hinter einem Spielernamen angezeigt, wenn es mehr als einen Spieler mit dem selben Namen gibt. Hierfür wurde eine neue Spalte in der Datenbank hinzugefügt. Diese wird initial auf 0 gesetzt und stellt die Kennzeichnung dar, der wie vielte Spieler mit dem gleichen Namen dieser ist.

Änderungen

  • Neue Spalte nameCount in der Player-Tabelle
  • CRUD-Operationen für nameCount-Spalte implementiert
  • Aktualisierungs-Logik in addPlayer, addPlayersAsList und updatePlayerName implementiert
  • Tests für Operationen implementiert
  • nameCount in allen UI-Tiles implementiert
  • Update-Logik bei Spieler-Erstellung in PlayerSelection-Widget implementiert
  • StatisticTiles für nameCount-Darstellung angepasst

Zusätzliche Anmerkungen

Gerne Rückmeldung zum visuellen, ob das so gut aussieht bzw. oder ob Ihr andere Ideen für die Darstellung hättet :>

### Zugehörige Issue(s) Closes #182 ### Beschreibung Im Interface wird jetzt ein kleines `#<num>` hinter einem Spielernamen angezeigt, wenn es mehr als einen Spieler mit dem selben Namen gibt. Hierfür wurde eine neue Spalte in der Datenbank hinzugefügt. Diese wird initial auf `0` gesetzt und stellt die Kennzeichnung dar, der wie vielte Spieler mit dem gleichen Namen dieser ist. | <img src="/attachments/5733beb6-c1b6-43b3-9672-a68ec7943fc5" width=400> | <img src="/attachments/57ba9ca5-c685-4070-b728-ce213772564d" width=400> | | --- | --- | | <img src="/attachments/e4f8b3ec-f6bd-41fd-8e96-b76ac01c78dd" width=400> | <img src="/attachments/1c03bfdd-c796-459d-8b61-75043232abff" width=400> | ### Änderungen - [x] Neue Spalte `nameCount` in der Player-Tabelle - [x] CRUD-Operationen für `nameCount`-Spalte implementiert - [x] Aktualisierungs-Logik in addPlayer, addPlayersAsList und updatePlayerName implementiert - [x] Tests für Operationen implementiert - [x] `nameCount` in allen UI-Tiles implementiert - [x] Update-Logik bei Spieler-Erstellung in `PlayerSelection`-Widget implementiert - [x] `StatisticTile`s für `nameCount`-Darstellung angepasst ### Zusätzliche Anmerkungen Gerne Rückmeldung zum visuellen, ob das so gut aussieht bzw. oder ob Ihr andere Ideen für die Darstellung hättet :>
Simulator Screenshot - iPhone 17 - 2026-04-19 at 23.32.19.png
76 KiB
Simulator Screenshot - iPhone 17 - 2026-04-19 at 23.33.16.png
56 KiB
Simulator Screenshot - iPhone 17 - 2026-04-20 at 15.50.45.png
78 KiB
Simulator Screenshot - iPhone 17 - 2026-04-20 at 16.38.43.png
78 KiB
flixcoo added this to the Alpha milestone 2026-04-19 21:34:54 +00:00
flixcoo added the
PR
Ready for Review
2
 label 2026-04-19 21:34:54 +00:00
flixcoo self-assigned this 2026-04-19 21:34:54 +00:00
flixcoo added 4 commits 2026-04-19 21:34:54 +00:00
Added database functionality + tests a1398623b0
Added ui implementation 9a2afbfd3b
Fixed tests 653b85d28d
Implemented name count update in player selection
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 43s
Details
Pull Request Pipeline / lint (pull_request) Successful in 49s
Details
fcf845af4d
flixcoo requested review from Owners 2026-04-19 21:34:54 +00:00
flixcoo added 1 commit 2026-04-19 21:41:16 +00:00
Added case for not fetching a player
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 43s
Details
Pull Request Pipeline / lint (pull_request) Successful in 46s
Details
9a0386f22d
flixcoo commented 2026-04-19 21:49:18 +00:00
Author
Owner
Copy Link

Man könnte beim Design z.B. auch so was in der Art machen:

Man könnte beim Design z.B. auch so was in der Art machen: <img src="/attachments/b35bef4a-0218-4e9b-8ce8-14fcede94b06" width=400>
Simulator Screenshot - iPhone 17 - 2026-04-19 at 23.48.32.png
76 KiB
flixcoo added 1 commit 2026-04-20 14:39:37 +00:00
Added nameCount to statistic tiles
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 43s
Details
Pull Request Pipeline / lint (pull_request) Successful in 50s
Details
4e97f6723a
gelbeinhalb reviewed 2026-04-21 08:40:51 +00:00
gelbeinhalb left a comment
Owner
Copy Link

Man könnte beim Design z.B. auch so was in der Art machen:

Ich finde das ohne diesen grauen Rand besser. Also so wie es oben in den screenshots ist

> Man könnte beim Design z.B. auch so was in der Art machen: > <img src="/attachments/b35bef4a-0218-4e9b-8ce8-14fcede94b06" width=400> Ich finde das ohne diesen grauen Rand besser. Also so wie es oben in den screenshots ist
👍 1
gelbeinhalb approved these changes 2026-04-21 08:43:34 +00:00
Dismissed
gelbeinhalb dismissed gelbeinhalb's review 2026-04-21 08:43:53 +00:00
Reason:

upsi

gelbeinhalb requested review from gelbeinhalb 2026-04-21 08:44:01 +00:00
gelbeinhalb commented 2026-04-21 08:44:19 +00:00
Owner
Copy Link

muss ich mir nochmal genauer angucken. bin noch in der uni

muss ich mir nochmal genauer angucken. bin noch in der uni
👍 1
flixcoo commented 2026-04-21 09:32:00 +00:00
Author
Owner
Copy Link

Ich finde das ohne diesen grauen Rand besser. Also so wie es oben in den screenshots ist

Ja checke, wollte das so ein bisschen an das Label Design von Gitea anlehnen

> Ich finde das ohne diesen grauen Rand besser. Also so wie es oben in den screenshots ist Ja checke, wollte das so ein bisschen an das Label Design von Gitea anlehnen
flixcoo added 2 commits 2026-04-21 10:56:43 +00:00
Merge pull request 'MVP' (#141) from development into main f0379e97c7 
Reviewed-on: #141
Reviewed-by: gelbeinhalb <spam@yannick-weigert.de>
Merge branch 'main' into enhancement/182-kennzeichnung-fuer-doppelte-spielernamen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 48s
Details
Pull Request Pipeline / lint (pull_request) Successful in 49s
Details
381c1ae23a
flixcoo added 1 commit 2026-04-21 10:59:37 +00:00
Merge branch 'development' into enhancement/182-kennzeichnung-fuer-doppelte-spielernamen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 42s
Details
Pull Request Pipeline / lint (pull_request) Successful in 50s
Details
487dbf944c
flixcoo scheduled this pull request to auto merge when all checks succeed 2026-04-23 15:13:55 +00:00
gelbeinhalb approved these changes 2026-04-24 09:18:42 +00:00
flixcoo merged commit 3140c9310b into development 2026-04-24 09:18:44 +00:00
flixcoo deleted branch enhancement/182-kennzeichnung-fuer-doppelte-spielernamen 2026-04-24 09:18:44 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
gelbeinhalb
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
Proposal
This issue contains a proposal for a new feature, change, or improvement. It is open for discussion and feedback before any decision is made.
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
Hotfix
Task
Refactoring
Tasks that only involve refactoring and do not add any new functionality
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
Technical Debt
Issues related to code that needs to be refactored, cleaned up, or improved due to shortcuts or outdated implementations and to keep the codebase maintainable.
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 Alpha
Assignees
Clear assignees
actions flixcoo gelbeinhalb sneeex
No Assignees flixcoo
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#200
Delete Branch "enhancement/182-kennzeichnung-fuer-doppelte-spielernamen"

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?