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

WIP: Implementierung von deleted Attribut #204

Draft
gelbeinhalb wants to merge 16 commits from bug/195-datenbank-onDelete-ueberpruefen into development
pull from: bug/195-datenbank-onDelete-ueberpruefen
merge into: liquid-development:development
Conversation 12 Commits 16 Files Changed 17 +616 -83
gelbeinhalb commented 2026-04-30 09:12:14 +00:00
Owner
Copy Link

Zugehörige Issue(s)

Closes #195

Beschreibung

@sneeex hat in #195 das onDelete Verhalten der Datenbank angesprochen und zur Überprüfung markiert.
Im Rahmen dieses PRs wurde das Verhalten erneut geprüft. Dabei hat sich gezeigt, dass die bereits von ihm vorgenommene Korrektur ausreichend ist und kein weiteres unerwünschtes Verhalten besteht.

Zusätzlich ist aufgefallen, dass ursprünglich ein deleted-Attribut vorgesehen war, jedoch in #88 versehentlich nicht implementiert wurde.

Änderungen

  • Verhalten von onDelete überprüft
  • Implementieren eines deleted-Attributs

Zusätzliche Anmerkungen

-

### Zugehörige Issue(s) Closes #195 ### Beschreibung @sneeex hat in #195 das onDelete Verhalten der Datenbank angesprochen und zur Überprüfung markiert. Im Rahmen dieses PRs wurde das Verhalten erneut geprüft. Dabei hat sich gezeigt, dass die bereits von ihm vorgenommene Korrektur ausreichend ist und kein weiteres unerwünschtes Verhalten besteht. Zusätzlich ist aufgefallen, dass ursprünglich ein `deleted`-Attribut vorgesehen war, jedoch in #88 versehentlich nicht implementiert wurde. ### Änderungen - [x] Verhalten von onDelete überprüft - [x] Implementieren eines `deleted`-Attributs ### Zusätzliche Anmerkungen \-
gelbeinhalb added this to the Alpha milestone 2026-04-30 09:12:14 +00:00
gelbeinhalb added the
Workflow
In Progress
3
 label 2026-04-30 09:12:14 +00:00
gelbeinhalb self-assigned this 2026-04-30 09:12:14 +00:00
gelbeinhalb added 1 commit 2026-04-30 09:12:14 +00:00
add soft delete parameter
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 44s
Details
Pull Request Pipeline / lint (pull_request) Successful in 47s
Details
8bd53a69c3
gelbeinhalb added 2 commits 2026-04-30 09:56:23 +00:00
Merge remote-tracking branch 'origin/development' into bug/195-datenbank-onDelete-ueberpruefen 1ec1df3514 
# Conflicts:
#	lib/data/db/tables/player_table.dart
add deleted attribute
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 45s
Details
Pull Request Pipeline / lint (pull_request) Successful in 46s
Details
66e657235a
gelbeinhalb added 1 commit 2026-05-05 08:35:11 +00:00
add deleted to schema
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 43s
Details
Pull Request Pipeline / lint (pull_request) Successful in 50s
Details
421fe1a2bf
flixcoo removed the
Workflow
In Progress
3
 label 2026-05-09 15:44:40 +00:00
gelbeinhalb started working 2026-05-12 18:00:33 +00:00
gelbeinhalb worked for 5 seconds 2026-05-12 18:00:38 +00:00
gelbeinhalb deleted spent time 2026-05-12 18:00:42 +00:00
- 5 seconds
gelbeinhalb added 1 commit 2026-05-12 18:20:03 +00:00
Merge remote-tracking branch 'origin/development' into bug/195-datenbank-onDelete-ueberpruefen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 53s
Details
81b73beeef 
# Conflicts:
#	assets/schema.json
#	lib/data/db/tables/player_match_table.dart
#	lib/data/models/game.dart
gelbeinhalb added 5 commits 2026-05-12 18:35:14 +00:00
remove useless deleted attributes d3c9827529
add deleted to schema b3b3def07a
deleted games get ignored when includeDeleted is false 36ad2ab446
deleted groups get ignored when includeDeleted is false 8f4254748b
deleted matches get ignored when includeDeleted is false
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
0870f418d5
gelbeinhalb added 2 commits 2026-05-12 19:02:16 +00:00
deleted players get ignored when includeDeleted is false b1934e64bd
deleted teams get ignored when includeDeleted is false
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 46s
Details
Pull Request Pipeline / lint (pull_request) Successful in 54s
Details
a6e0971208
gelbeinhalb added 2 commits 2026-05-12 19:47:44 +00:00
regenerate database.g.dart c674c8e248
deleted is not required anymore
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 47s
Details
Pull Request Pipeline / lint (pull_request) Successful in 55s
Details
e6e065ed44
gelbeinhalb added the
PR
Ready for Review
2
Workflow
In Progress
3
 labels 2026-05-12 20:07:32 +00:00
gelbeinhalb requested review from Owners 2026-05-12 20:07:36 +00:00
flixcoo requested changes 2026-05-12 20:20:46 +00:00
Dismissed
flixcoo left a comment
Owner
Copy Link

Beschreibung fehlt.

Beschreibung fehlt.
gelbeinhalb commented 2026-05-12 20:53:30 +00:00
Author
Owner
Copy Link

fettsack

fettsack
sneeex requested changes 2026-05-12 20:57:15 +00:00
sneeex left a comment
Owner
Copy Link

Label in Progress und Ready for Review und WIP im Titel macht keinen sinn

Label in Progress und Ready for Review und WIP im Titel macht keinen sinn
gelbeinhalb commented 2026-05-12 21:03:54 +00:00
Author
Owner
Copy Link

jungs ihr seid solche nervigen Menschen

jungs ihr seid solche nervigen Menschen
😆 2
gelbeinhalb removed the
Workflow
In Progress
3
 label 2026-05-12 21:04:01 +00:00
gelbeinhalb commented 2026-05-12 21:04:07 +00:00
Author
Owner
Copy Link

Jetzt besser?

Jetzt besser?
👍 2
gelbeinhalb changed title from WIP: Implementierung von deleted Attribut to Implementierung von deleted Attribut 2026-05-12 21:04:40 +00:00
gelbeinhalb requested review from flixcoo 2026-05-12 21:06:28 +00:00
gelbeinhalb requested review from sneeex 2026-05-12 21:06:30 +00:00
flixcoo requested changes 2026-05-12 21:16:57 +00:00
Dismissed
flixcoo left a comment
Owner
Copy Link

Was genau soll denn die aktuelle Implementierung von der deleted-Flag bringen bzw. vorbereiten? Die einzige Stelle (wie im Meeting besprochen) wo dieses Attribut Sinn macht sind die Spieler. Bei allen anderen sehe ich den Sinn dahinter nicht ganz.

  1. Matches sollen vollständig gelöscht werden
  2. Games löschen Matches mit, was ja auch intendet ist. Hier könnte man es so ändern, dass diese nicht gelöscht werden um die Matches zu behalten.
  3. Teams sind innerhalb eines Matches und werden nicht getrennt behandelt, sind also equvalent zu 1.
  4. Groups sind ja nur Aggregationen von Spielern. Da diese eh immer dem Match zugeteilt sind und eine Gruppe vor allem für die Statistiken zum Zuordnen wichtig sind, brauchen diese m.M.n. auch keine Flag
  5. ScoreEntrys werden nie vom User abgegriffen und verhalten sich ebenfalls wie Teams, siehe 1.
Was genau soll denn die aktuelle Implementierung von der `deleted`-Flag bringen bzw. vorbereiten? Die einzige Stelle (wie im Meeting besprochen) wo dieses Attribut Sinn macht sind die Spieler. Bei allen anderen sehe ich den Sinn dahinter nicht ganz. 1. **Matches** sollen vollständig gelöscht werden 2. **Games** löschen Matches mit, was ja auch intendet ist. Hier könnte man es so ändern, dass diese nicht gelöscht werden um die Matches zu behalten. 3. **Teams** sind innerhalb eines Matches und werden nicht getrennt behandelt, sind also equvalent zu 1. 4. **Groups** sind ja nur Aggregationen von Spielern. Da diese eh immer dem Match zugeteilt sind und eine Gruppe vor allem für die Statistiken zum Zuordnen wichtig sind, brauchen diese m.M.n. auch keine Flag 5. **ScoreEntrys** werden nie vom User abgegriffen und verhalten sich ebenfalls wie Teams, siehe 1.
gelbeinhalb commented 2026-05-12 21:28:10 +00:00
Author
Owner
Copy Link

ja stimmt hast recht. Aber score entries haben auch garkeine deleted flag?

ja stimmt hast recht. Aber score entries haben auch garkeine deleted flag?
gelbeinhalb commented 2026-05-12 21:29:17 +00:00
Author
Owner
Copy Link

Ich hatte bei den ganzen Sachen mehr an eine Art Wiederherstellungsmöglichkeit gedacht, falls man mal Sachen ausversehen löscht.

Ich hatte bei den ganzen Sachen mehr an eine Art Wiederherstellungsmöglichkeit gedacht, falls man mal Sachen ausversehen löscht.
gelbeinhalb requested review from flixcoo 2026-05-12 21:29:32 +00:00
flixcoo added
PR
In Review
1
 and removed
PR
Ready for Review
2
 labels 2026-05-13 09:30:58 +00:00
sneeex commented 2026-05-13 10:47:24 +00:00
Owner
Copy Link

@flixcoo @gelbeinhalb ich würde glaube sagen, wir machen das nur für die player oder? weil dann noch irgendwie mit irgendwelchen wiederherstellungssachen zu dealen ist ja auch nervig, es ging ja nur darum, dass gelöschte spieler trotzdem noch angezeigt werden können in alten matches oder nicht?

@flixcoo @gelbeinhalb ich würde glaube sagen, wir machen das nur für die player oder? weil dann noch irgendwie mit irgendwelchen wiederherstellungssachen zu dealen ist ja auch nervig, es ging ja nur darum, dass gelöschte spieler trotzdem noch angezeigt werden können in alten matches oder nicht?
flixcoo commented 2026-05-15 09:29:04 +00:00
Owner
Copy Link

ja stimmt hast recht. Aber score entries haben auch garkeine deleted flag?

grafik.png

Bildschirmfoto 2026-05-15 um 11.27.59.png

> ja stimmt hast recht. Aber score entries haben auch garkeine deleted flag? ![grafik.png](/attachments/eb6e3a3e-4c81-4243-98ee-d752e7621c3c) ![Bildschirmfoto 2026-05-15 um 11.27.59.png](/attachments/0540c431-3a38-453d-8791-e9e4eca8e656)
Bildschirmfoto 2026-05-15 um 11.27.59.png
287 KiB
grafik.png
54 KiB
flixcoo commented 2026-05-15 09:29:21 +00:00
Owner
Copy Link

@flixcoo @gelbeinhalb ich würde glaube sagen, wir machen das nur für die player oder? weil dann noch irgendwie mit irgendwelchen wiederherstellungssachen zu dealen ist ja auch nervig, es ging ja nur darum, dass gelöschte spieler trotzdem noch angezeigt werden können in alten matches oder nicht?

lass darüber mal im meeting diskutieren

> @flixcoo @gelbeinhalb ich würde glaube sagen, wir machen das nur für die player oder? weil dann noch irgendwie mit irgendwelchen wiederherstellungssachen zu dealen ist ja auch nervig, es ging ja nur darum, dass gelöschte spieler trotzdem noch angezeigt werden können in alten matches oder nicht? lass darüber mal im meeting diskutieren
sneeex added 1 commit 2026-05-21 19:55:44 +00:00
Merge branch 'development' into bug/195-datenbank-onDelete-ueberpruefen
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 45s
Details
Pull Request Pipeline / lint (pull_request) Successful in 53s
Details
4c77eaa8c9
flixcoo requested changes 2026-05-22 18:08:04 +00:00
flixcoo left a comment
Owner
Copy Link

PR fixen oder neu aufsetzen und richtige Änderungen implementieren

PR fixen oder neu aufsetzen und richtige Änderungen implementieren
flixcoo changed title from Implementierung von deleted Attribut to WIP: Implementierung von deleted Attribut 2026-05-22 18:08:19 +00:00
flixcoo removed the
PR
In Review
1
 label 2026-05-22 18:08:28 +00:00
flixcoo added 1 commit 2026-05-23 14:15:07 +00:00
Merge branch 'development' into bug/195-datenbank-onDelete-ueberpruefen
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 45s
Details
Pull Request Pipeline / test (pull_request) Successful in 47s
Details
Pull Request Pipeline / localizations (pull_request) Successful in 25s
Details
13b3d2cad9
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 45s
Required
Details
Pull Request Pipeline / test (pull_request) Successful in 47s
Required
Details
Pull Request Pipeline / localizations (pull_request) Successful in 25s
Required
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin bug/195-datenbank-onDelete-ueberpruefen:bug/195-datenbank-onDelete-ueberpruefen
git checkout bug/195-datenbank-onDelete-ueberpruefen
Sign in to join this conversation.
Reviewers
No Reviewers
sneeex
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
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
Milestone
No items
No Milestone Alpha
Assignees
Clear assignees
actions flixcoo gelbeinhalb sneeex
No Assignees gelbeinhalb
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#204
Delete Branch "bug/195-datenbank-onDelete-ueberpruefen"

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?