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

Bugfix bei Spieleranzeige und Vereinfachung der Keine-Player-Logik #66

Merged
flixcoo merged 1 commits from bug/60-CreateGroupView-erstellter-Spieler-erscheint-nicht-in-alle-Spieler into development 2025-11-23 21:41:08 +00:00
Conversation 1 Commits 1 Files Changed 1 +7 -6
sneeex commented 2025-11-23 21:24:15 +00:00
Owner
Copy Link

Bugfix bei Spieleranzeige und Vereinfachung der Keine-Player-Logik

Zugehörige Issue(s):
Closes #60

Beschreibung

Dieser PR führt die booleschen Variablen doneLoading und snapshotDataEmpty ein, um die Bedingungen für die Anzeige der Empty-State-Nachricht zu vereinfachen. Er korrigiert außerdem die Logik so, dass die Meldung „Keine Spieler gefunden“ nur angezeigt wird, wenn sowohl der Snapshot als auch die lokale allPlayers-Liste leer sind, und entfernt dabei die bisherige Abhängigkeit von selectedPlayers.isEmpty, weil wenn allPlayers Leer ist, auch die selected Players leer sein muss.

Änderungen

  • Führt doneLoading & snapshotDataEmpty zur Vereinfachung und besseren Readability ein
  • Fixxt den Bug, wo ein erstellter Spieler nicht in alle Spieler angezeigt wird.
# Bugfix bei Spieleranzeige und Vereinfachung der Keine-Player-Logik **Zugehörige Issue(s):** Closes #60 ## Beschreibung Dieser PR führt die booleschen Variablen doneLoading und snapshotDataEmpty ein, um die Bedingungen für die Anzeige der Empty-State-Nachricht zu vereinfachen. Er korrigiert außerdem die Logik so, dass die Meldung „Keine Spieler gefunden“ nur angezeigt wird, wenn sowohl der Snapshot als auch die lokale allPlayers-Liste leer sind, und entfernt dabei die bisherige Abhängigkeit von selectedPlayers.isEmpty, weil wenn allPlayers Leer ist, auch die selected Players leer sein muss. ## Änderungen - [x] Führt doneLoading & snapshotDataEmpty zur Vereinfachung und besseren Readability ein - [x] Fixxt den Bug, wo ein erstellter Spieler nicht in alle Spieler angezeigt wird.
sneeex added this to the Minimum Viable Product (MVP) milestone 2025-11-23 21:24:15 +00:00
sneeex added the
PR
Ready for Review
2
 label 2025-11-23 21:24:15 +00:00
sneeex self-assigned this 2025-11-23 21:24:15 +00:00
sneeex added 1 commit 2025-11-23 21:24:15 +00:00
refactor empty state logic in CreateGroupView
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m22s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m24s
Details
974d6e9b56 
The diff introduces boolean variables `doneLoading` and `snapshotDataEmpty` to simplify the conditional check for displaying the empty state message. It specifically fixes the logic to correctly show the "No players found" message when both the snapshot and the local `allPlayers` list are empty, removing the dependency on `selectedPlayers.isEmpty`.
sneeex requested review from Owners 2025-11-23 21:24:15 +00:00
flixcoo approved these changes 2025-11-23 21:40:52 +00:00
flixcoo merged commit 0b500b5248 into development 2025-11-23 21:41:08 +00:00
flixcoo deleted branch bug/60-CreateGroupView-erstellter-Spieler-erscheint-nicht-in-alle-Spieler 2025-11-23 21:41:08 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#66
Delete Branch "bug/60-CreateGroupView-erstellter-Spieler-erscheint-nicht-in-alle-Spieler"

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?