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

JSON Import für Testdaten & Funktion zum Löschen aller Daten #33

Merged
flixcoo merged 44 commits from feature/31-json-import-fuer-testdaten into development 2025-11-22 16:47:15 +00:00
Conversation 10 Commits 44 Files Changed 22 +1029 -305
flixcoo commented 2025-11-17 22:26:49 +00:00
Owner
Copy Link

JSON Import für Testdaten

Zugehörige Issue(s):
Closes #31 #32

Beschreibung

Die Spieldaten können nun über die Einstellungen exportiert werden. Importierte Spieldaten werden über ein JSON-Schema überprüft und der Datenbank hinzugefügt. Ein Button der alle Daten der Datenbank löscht wurde hinzugefügt.

Änderungen

  • SettingsListTile implementiert
  • SettingsView Layout grob implementiert
  • Import & Export implementiert
  • Alle Daten löschen implementiert
  • Datenbank angepasst
  • Methoden addGames(), AddGroups() und addPlayers() implementiert
  • JSON-Schema-Überprüfung implementiert

Zusätzliche Anmerkungen

Aufgrund von Problemen beim kaskadierenden Löschen musste die Datenbank mit einem Custom Statement angepasst werden.

# JSON Import für Testdaten **Zugehörige Issue(s):** Closes #31 #32 ## Beschreibung Die Spieldaten können nun über die Einstellungen exportiert werden. Importierte Spieldaten werden über ein JSON-Schema überprüft und der Datenbank hinzugefügt. Ein Button der alle Daten der Datenbank löscht wurde hinzugefügt. ## Änderungen - `SettingsListTile` implementiert - `SettingsView` Layout grob implementiert - Import & Export implementiert - Alle Daten löschen implementiert - Datenbank angepasst - Methoden `addGames()`, `AddGroups()` und `addPlayers()` implementiert - JSON-Schema-Überprüfung implementiert ## Zusätzliche Anmerkungen Aufgrund von Problemen beim kaskadierenden Löschen musste die Datenbank mit einem Custom Statement angepasst werden.
flixcoo added this to the Minimum Viable Product (MVP) milestone 2025-11-17 22:26:49 +00:00
flixcoo added the
Priority
High
2
 label 2025-11-17 22:26:49 +00:00
flixcoo self-assigned this 2025-11-17 22:26:49 +00:00
flixcoo added 4 commits 2025-11-17 22:26:49 +00:00
Implemented new SettingsListTile 3ca081419b
Moved game tile in tiles folder 62acc87e0e
Created new layout for settings view 2076e45fd5
Added function headers 178aaa9643
sneeex commented 2025-11-18 07:55:19 +00:00
Owner
Copy Link

grafik.png

schrift muss auf jeden fall kleiner und warum ist das "Menu" auf einmal nicht mehr in der AppBar oben?
Ist ja sonst überall so

![grafik.png](/attachments/fef1de2e-d763-44a9-942c-5eb71bae3419) schrift muss auf jeden fall kleiner und warum ist das "Menu" auf einmal nicht mehr in der AppBar oben? Ist ja sonst überall so
grafik.png
43 KiB
sneeex commented 2025-11-18 07:55:48 +00:00
Owner
Copy Link

aber sieht cool aus

aber sieht cool aus
flixcoo commented 2025-11-18 08:13:46 +00:00
Author
Owner
Copy Link

grafik.png

schrift muss auf jeden fall kleiner und warum ist das "Menu" auf einmal nicht mehr in der AppBar oben?
Ist ja sonst überall so

Also 1. chill doch ich bin doch noch garnicht fertig
2. Wollte keine AppBar haben weil das dadurch weniger Android Like aussieht, meine Inspiration war das:

Ich find die DönerCheck App hat halt n gutes Hybrides Design und wollte mich daran bisschen orientieren

> ![grafik.png](/attachments/fef1de2e-d763-44a9-942c-5eb71bae3419) > > schrift muss auf jeden fall kleiner und warum ist das "Menu" auf einmal nicht mehr in der AppBar oben? > Ist ja sonst überall so Also 1. chill doch ich bin doch noch garnicht fertig 2. Wollte keine AppBar haben weil das dadurch weniger Android Like aussieht, meine Inspiration war das: <img height="500" src="/attachments/c6cdb6ea-0162-4dae-8122-094f2b8d60bc"> Ich find die DönerCheck App hat halt n gutes Hybrides Design und wollte mich daran bisschen orientieren
grafik.png
486 KiB
😕 2
flixcoo added 2 commits 2025-11-18 21:50:06 +00:00
Fixed gesture detector area a8a81c2151
Merge remote-tracking branch 'refs/remotes/origin/development' into feature/31-json-import-fuer-testdaten 07d623d963
sneeex changed title from WIP: JSON Import für Testdaten to WIP: JSON Import für Testdaten & Funktion zum Löschen aller Daten 2025-11-18 22:03:25 +00:00
flixcoo added a new dependency 2025-11-18 22:54:55 +00:00
#36 Timestamp zu allen Objekten hinzufügen
flixcoo added 10 commits 2025-11-18 23:32:27 +00:00
Added fromJson, toJson d86de09042
Added json schema 2da2e28cb6
Fixed toJson methods fd86f5193f
Added methods for deleting all entities 08fcaa35ee
Added schema.json 42ce69f4d3
Added custom statement for cascade deleting 69c95ca672
Changed table column because of importing issues f6ebda7984
Adjusted attributes to table definition 5dcd0826bd
First version of settings view f2a749cb0f
Refactored whole export/import methods in DataTransferService 82e28b7509
flixcoo requested review from gelbeinhalb 2025-11-18 23:32:44 +00:00
flixcoo requested review from sneeex 2025-11-18 23:32:44 +00:00
flixcoo added the
PR
Ready for Review
2
 label 2025-11-18 23:32:48 +00:00
flixcoo changed title from WIP: JSON Import für Testdaten & Funktion zum Löschen aller Daten to JSON Import für Testdaten & Funktion zum Löschen aller Daten 2025-11-18 23:33:14 +00:00
flixcoo added a new dependency 2025-11-19 00:06:28 +00:00
#29 Timestamp zu allen Objekten hinzufügen
flixcoo added a new dependency 2025-11-19 00:13:51 +00:00
#17 Actions aufsetzen
flixcoo removed a dependency 2025-11-19 00:13:58 +00:00
#17 Actions aufsetzen
flixcoo added 1 commit 2025-11-19 08:44:55 +00:00
Added documentation and feedback in snackbar 322c51a764
sneeex requested changes 2025-11-19 17:44:40 +00:00
Dismissed
sneeex left a comment
Owner
Copy Link

createdAt field hinzufügen zu jeder group, game und player.
Davor muss #36 approved und gemerged werden.

createdAt field hinzufügen zu jeder group, game und player. Davor muss #36 approved und gemerged werden.
flixcoo added 1 commit 2025-11-19 18:23:14 +00:00
Merge remote-tracking branch 'origin/development' into feature/31-json-import-fuer-testdaten cf834e636e 
# Conflicts:
#	lib/presentation/views/main_menu/home_view.dart
flixcoo added 1 commit 2025-11-19 18:26:39 +00:00
Moved ImportStatus & ExportStatus to enums.dart 87b1a7d57f
sneeex added
PR
In Review
1
 and removed
PR
Ready for Review
2
 labels 2025-11-19 18:59:39 +00:00
flixcoo added 13 commits 2025-11-19 19:22:00 +00:00
added createdAt timestamp to Group, Game, and Player DTOs 67c8a7e181
regenerated database.g.dart 8cc898cad6
add createdAt column to game, group and player tables and DAOs 2ee8edcf9b
use clock.now() instead of DateTime.now() for DTO creation timestamps 19c99eef9c
verify createdAt timestamps in database tests using mocked clock 75c6f4e01c
add clock dependency to pubspec.yaml 8150b42dba
created all objects in setup() funktion to avoid redundant withClock c76e193b4d
removed uneccessary withClock 3b6a914022
Merge branch 'development' into feature/29-timestamp-zu-allen-objekten-hinzufügen 186d8cf5c9
Merge remote-tracking branch 'origin/feature/29-timestamp-zu-allen-objekten-hinzufügen' into feature/31-json-import-fuer-testdaten 3e3def0bde 
# Conflicts:
#	lib/data/dao/game_dao.dart
#	lib/data/db/database.g.dart
#	lib/data/db/tables/game_table.dart
#	lib/data/dto/game.dart
#	pubspec.yaml
Added createdAt attribute in dto classes and json schema 9434282ed1
Added insert mode f7073a83a4
Added methods for inserting list of players, groups, games 412cfff9f5
flixcoo requested review from sneeex 2025-11-19 19:22:03 +00:00
flixcoo added 1 commit 2025-11-19 19:22:27 +00:00
Merge branch 'development' into feature/31-json-import-fuer-testdaten 061a927324
flixcoo removed a dependency 2025-11-19 19:23:24 +00:00
#36 Timestamp zu allen Objekten hinzufügen
flixcoo added 1 commit 2025-11-19 19:27:34 +00:00
Merge branch 'development' into feature/31-json-import-fuer-testdaten 682c1d269d
flixcoo added 1 commit 2025-11-19 19:35:42 +00:00
Merge branch 'development' into feature/31-json-import-fuer-testdaten
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m4s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m13s
Details
17dabb773d
sneeex requested changes 2025-11-19 19:42:01 +00:00
sneeex left a comment
Owner
Copy Link

@flixcoo test fehlgeschlagen hier

@flixcoo test fehlgeschlagen hier
sneeex requested changes 2025-11-19 19:52:27 +00:00
lib/presentation/views/main_menu/settings_view.dart Outdated
@@ -11,0 +80,4 @@
await DataTransferService.getAppDataAsJson(context);
final result = await DataTransferService.exportData(
json,
'exported_data',
sneeex commented 2025-11-19 19:48:29 +00:00
Owner
Copy Link

name exported data blöd, lieber game_tracker_data oder sowas

name exported data blöd, lieber game_tracker_data oder sowas
flixcoo marked this conversation as resolved
lib/presentation/views/main_menu/settings_view.dart Outdated
@@ -11,0 +108,4 @@
context: context,
message: 'Data successfully deleted',
);
},
sneeex commented 2025-11-19 19:46:18 +00:00
Owner
Copy Link
  • bei delete all data auf jeden fall ein Dialog mit bestätigen, habe gerade ausversehen alle daten gelöscht xD, dachte da kommt sowas
  • musst die jeweilige page dann neuladen, updated aktuell nicht
- bei delete all data auf jeden fall ein Dialog mit bestätigen, habe gerade ausversehen alle daten gelöscht xD, dachte da kommt sowas - musst die jeweilige page dann neuladen, updated aktuell nicht
flixcoo marked this conversation as resolved
flixcoo added 1 commit 2025-11-19 20:13:58 +00:00
Added missing await
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
f7c1d6e975
sneeex requested changes 2025-11-19 20:34:39 +00:00
assets/schema.json Outdated
@@ -0,0 +23,4 @@
"type": "null"
},
"group": {
"type": "null"
sneeex commented 2025-11-19 20:34:35 +00:00
Owner
Copy Link

alles was null ist, ist forced null, d.h. es muss null sein, ist hier aber ja nicht richtig

alles was null ist, ist forced null, d.h. es muss null sein, ist hier aber ja nicht richtig
flixcoo marked this conversation as resolved
flixcoo added 4 commits 2025-11-19 20:42:16 +00:00
changed export file name cf71b40718
Tabs update themselves after settings view a8d4e640cf
Added dialog 822bc03c83
Updated schema
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m5s
Details
f40a9ad09b
flixcoo added 1 commit 2025-11-19 20:51:17 +00:00
Corrected schema again
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m3s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m14s
Details
45650133a7
sneeex removed a dependency 2025-11-19 21:00:08 +00:00
#29 Timestamp zu allen Objekten hinzufügen
flixcoo commented 2025-11-19 21:30:06 +00:00
Author
Owner
Copy Link

Immer noch Probleme beim exportieren, vielleicht werden irgendwo daten überschrieben?

Immer noch Probleme beim exportieren, vielleicht werden irgendwo daten überschrieben?
flixcoo changed title from JSON Import für Testdaten & Funktion zum Löschen aller Daten to WIP: JSON Import für Testdaten & Funktion zum Löschen aller Daten 2025-11-20 17:30:01 +00:00
flixcoo added 2 commits 2025-11-20 22:00:29 +00:00
Fixed error in getAllGames method a61818dd77
Merge branch 'bug/37-getallgames-returnt-nicht-alle-attribute-von-der-game-klasse' into feature/31-json-import-fuer-testdaten
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m4s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
29a3e77fc4
flixcoo added 4 commits 2025-11-20 22:51:31 +00:00
Updated insert modes 72067863c2
Overhauled tests
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m8s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
Details
89b3f1ff69
Merge branch 'bug/37-getallgames-returnt-nicht-alle-attribute-von-der-game-klasse' into feature/31-json-import-fuer-testdaten e512cbbf95
Added methods for multiple inserts to tests
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
8e63a01705
flixcoo added 12 commits 2025-11-21 13:01:50 +00:00
Added methods of todos 31589855f2
Refactoring 6055eb63a8
Refactoring b21ca54672
Refactored tests in own files
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m5s
Details
961c6bb679
Added missing test fe9239ee02
Fixed test 229750ffcf
Annotation for missing test & method
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m1s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m4s
Details
32f3f68da9
Added missing methods
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
Details
e15f5d163d
Added iteration for multiple items test
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
d948f2f13d
Renamed variables to be consistent
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
8c05385203
Merge branch 'bug/37-getallgames-returnt-nicht-alle-attribute-von-der-game-klasse' into feature/31-json-import-fuer-testdaten 78d530ddd5 
# Conflicts:
#	lib/data/dao/player_group_dao.dart
#	test/db_tests/game_test.dart
#	test/db_tests/group_test.dart
#	test/db_tests/player_test.dart
Added batch insert methods to tests
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m9s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
Details
51722eb7fd
flixcoo added 1 commit 2025-11-21 13:17:43 +00:00
Added missing await
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m4s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m5s
Details
c56663d15e
flixcoo added 3 commits 2025-11-21 13:26:27 +00:00
Added missing awaits
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m7s
Details
dbb52cfc48
Typo
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 2m25s
Details
Pull Request Pipeline / test (pull_request) Successful in 2m25s
Details
ab250e2df4
Merge branch 'bug/37-getallgames-returnt-nicht-alle-attribute-von-der-game-klasse' into feature/31-json-import-fuer-testdaten
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m21s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m21s
Details
7cd53aa695 
# Conflicts:
#	test/db_tests/player_test.dart
flixcoo added 3 commits 2025-11-21 23:48:36 +00:00
Fixed addGroups method 1b3334f3e0
Fixed addGames method 70307016b3
Moved method validateJsonSchema()
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m5s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m6s
Details
2ebd4274f0
flixcoo requested review from sneeex 2025-11-21 23:51:05 +00:00
flixcoo changed title from WIP: JSON Import für Testdaten & Funktion zum Löschen aller Daten to JSON Import für Testdaten & Funktion zum Löschen aller Daten 2025-11-21 23:51:13 +00:00
flixcoo commented 2025-11-22 00:11:44 +00:00
Author
Owner
Copy Link

Beispieldaten, welche ab jetzt genutzt werden können und ggf. erweitert werden können sind im Discord

Beispieldaten, welche ab jetzt genutzt werden können und ggf. erweitert werden können sind im [Discord](https://discord.com/channels/921763177245605888/938410896299028570/1441581737733001288)
flixcoo added
PR
Ready for Review
2
 and removed
PR
In Review
1
 labels 2025-11-22 00:12:07 +00:00
flixcoo added 1 commit 2025-11-22 00:12:52 +00:00
Schema corrected
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m10s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m11s
Details
893eb91143
flixcoo added 1 commit 2025-11-22 15:44:19 +00:00
Merge branch 'development' into feature/31-json-import-fuer-testdaten
All checks were successful
Pull Request Pipeline / lint (pull_request) Successful in 2m16s
Details
Pull Request Pipeline / test (pull_request) Successful in 2m16s
Details
0593297fc3
gelbeinhalb approved these changes 2025-11-22 16:40:36 +00:00
gelbeinhalb left a comment
Owner
Copy Link

LGTM

lhm3ijxuv0f91.gif

LGTM ![lhm3ijxuv0f91.gif](/attachments/b4ff4aba-79a3-4412-9421-8c6ccf6b36d0)
lhm3ijxuv0f91.gif
983 KiB
😆 1
flixcoo merged commit 6af1df5fbc into development 2025-11-22 16:47:15 +00:00
flixcoo deleted branch feature/31-json-import-fuer-testdaten 2025-11-22 16:47:15 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
sneeex
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
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
Priority
High
2
Milestone
No items
No Milestone Minimum Viable Product (MVP)
Assignees
Clear assignees
flixcoo gelbeinhalb sneeex
No Assignees flixcoo
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#33
Delete Branch "feature/31-json-import-fuer-testdaten"

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?