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

Kennzeichnung für doppelte Spielernamen #200

Open
flixcoo wants to merge 5 commits from enhancement/182-kennzeichnung-fuer-doppelte-spielernamen into development
pull from: enhancement/182-kennzeichnung-fuer-doppelte-spielernamen
merge into: liquid-development:development
Conversation 1 Commits 5 Files Changed 13 +577 -117
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.

MatchTiles GroupTiles

Ä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
  • playerCount in allen UI-Tiles implementiert
  • Update-Logik bei Spieler-Erstellung in PlayerSelection-Widget implementiert

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. | `MatchTiles` | `GroupTiles` | | --- | --- | | <img src="/attachments/5733beb6-c1b6-43b3-9672-a68ec7943fc5" width=400> | <img src="/attachments/57ba9ca5-c685-4070-b728-ce213772564d" 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] `playerCount` in allen UI-Tiles implementiert - [x] Update-Logik bei Spieler-Erstellung in `PlayerSelection`-Widget implementiert ### 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
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
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 43s
Required
Details
Pull Request Pipeline / lint (pull_request) Successful in 46s
Required
Details
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin enhancement/182-kennzeichnung-fuer-doppelte-spielernamen:enhancement/182-kennzeichnung-fuer-doppelte-spielernamen
git checkout enhancement/182-kennzeichnung-fuer-doppelte-spielernamen
Sign in to join this conversation.
Reviewers
No Reviewers
liquid-development/Owners
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
1 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?