Vereins-IT/ezgg-lan-manager
Archived
8
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

WIP: feature/tournament #11

Closed
tcprod wants to merge 3 commits from feature/tournament into main
pull from: feature/tournament
merge into: :main
Conversation 1 Commits 3 Files Changed 7
+272 -48
tcprod commented 2025-02-10 15:56:21 +00:00
Owner
Copy Link
Copy Source
No description provided.
tcprod added 2 commits 2025-02-10 15:56:22 +00:00
add tournaments feature 3eb9d6ab8c
expand database schema and add blueprints for Tournament feature 1679f537de
sql/create_database.sql
@@ -1,15 +1,15 @@
CREATE DATABASE IF NOT EXISTS `ez_lan_manager` /*!40100 DEFAULT CHARACTER SET utf8mb4 COLLATE utf8mb4_general_ci */;
USE `ez_lan_manager`;
-- MySQL dump 10.13 Distrib 5.7.24, for Linux (x86_64)
-- MySQL dump 10.13 Distrib 8.0.41, for Win64 (x86_64)
Typhus commented 2025-02-10 16:09:10 +00:00
Owner

Ich weiß nicht wie sehr es mir gefällt hier die MySQL Version von 5 auf 8 zu heben.

Hast du kontrolliert was diese Änderung für uns bedeuted?

Ich weiß nicht wie sehr es mir gefällt hier die MySQL Version von 5 auf 8 zu heben. Hast du kontrolliert was diese Änderung für uns bedeuted?
sql/create_database.sql
@@ -126,0 +133,4 @@
/*!40101 SET @saved_cs_client = @@character_set_client */;
/*!50503 SET character_set_client = utf8mb4 */;
CREATE TABLE `tournament_participants` (
`tournament_id` int(11) NOT NULL,
Typhus commented 2025-02-10 16:10:50 +00:00
Owner

Hier fehlt die Referenz auf den FK

Hier fehlt die Referenz auf den FK
sql/create_database.sql
@@ -126,0 +134,4 @@
/*!50503 SET character_set_client = utf8mb4 */;
CREATE TABLE `tournament_participants` (
`tournament_id` int(11) NOT NULL,
`user_id` int(11) NOT NULL,
Typhus commented 2025-02-10 16:10:55 +00:00
Owner

Hier fehlt die Referenz auf den FK

Hier fehlt die Referenz auf den FK
sql/create_database.sql
@@ -126,0 +147,4 @@
DROP TABLE IF EXISTS `tournaments`;
/*!40101 SET @saved_cs_client = @@character_set_client */;
/*!50503 SET character_set_client = utf8mb4 */;
CREATE TABLE `tournaments` (
Typhus commented 2025-02-10 16:13:24 +00:00
Owner

Mir ist klar das wir die Turniere zu beginn simpel halten wollen. Allerdings fehlt uns bei dieser Datenstruktur die Möglichkeit Teams teilnehmen zu lassen.

Ich würde daher ein weiteres Feld einfügen das darlegt wie groß die Teams sind. Und das dann erstmal hart auf 1 setzen (für Einzelspieler).

Dann müssen wir, wenn wir dann Teams haben, nichtmehr so viel umschmeißen, und können als participant einen User ODER ein Team nehmen.

Mir ist klar das wir die Turniere zu beginn simpel halten wollen. Allerdings fehlt uns bei dieser Datenstruktur die Möglichkeit Teams teilnehmen zu lassen. Ich würde daher ein weiteres Feld einfügen das darlegt wie groß die Teams sind. Und das dann erstmal hart auf `1` setzen (für Einzelspieler). Dann müssen wir, wenn wir dann Teams haben, nichtmehr so viel umschmeißen, und können als `participant` einen User ODER ein Team nehmen.
src/ez_lan_manager/services/DatabaseService.py Outdated
@@ -788,3 +788,21 @@ class DatabaseService:
return await self.remove_profile_picture(user_id)
except Exception as e:
logger.warning(f"Error deleting user profile picture: {e}")
Typhus commented 2025-02-10 16:14:28 +00:00
Owner

2 Dinge die im Hinterkopf zu behalten sind:

  • Nicht vergessen hier später die Typen dran zu schreiben :)
  • Es braucht auch Methoden um Turniere anzulegen, zu löschen, die Anmeldung zu öffnen, zu schließen und die Einstellungen zu bearbeiten. Das kann man alles in einzelnen Methoden machen, oder du legst eine Datenklasse für Turniere an, die dann die Services beide verstehen.
2 Dinge die im Hinterkopf zu behalten sind: - Nicht vergessen hier später die Typen dran zu schreiben :) - Es braucht auch Methoden um Turniere anzulegen, zu löschen, die Anmeldung zu öffnen, zu schließen und die Einstellungen zu bearbeiten. Das kann man alles in einzelnen Methoden machen, oder du legst eine Datenklasse für Turniere an, die dann die Services beide verstehen.
tcprod commented 2025-02-10 16:21:56 +00:00
Author
Owner

Das Kommentar verstehe ich nicht? Ich würde erstmal so weiter machen, wie ich denke und wenn ich an dem Problem ankomme, kannst du es mir nochmal erklären.

Das Kommentar verstehe ich nicht? Ich würde erstmal so weiter machen, wie ich denke und wenn ich an dem Problem ankomme, kannst du es mir nochmal erklären.
Typhus commented 2025-02-10 20:49:42 +00:00
Owner

Wollte es nur erwähnt haben

Wollte es nur erwähnt haben
src/ez_lan_manager/services/TournamentService.py
@@ -0,0 +12,4 @@
pass
class TournamentService:
Typhus commented 2025-02-10 16:18:27 +00:00
Owner

Den Methoden in dieser Klasse fehlt es noch etwas an Condition-checking.

Den Methoden in dieser Klasse fehlt es noch etwas an Condition-checking.
tcprod commented 2025-02-10 16:22:30 +00:00
Author
Owner

Die Methoden können doch noch gar nichts 😄

Die Methoden können doch noch gar nichts 😄
tcprod added 1 commit 2025-02-18 11:25:47 +00:00
WIP d98482f540
Typhus closed this pull request 2026-01-29 05:07:16 +00:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
Typhus
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
betawurst Catering-PC dusker MiniTT Orga-PC system (System Benutzer) tcprod Typhus (David Rodenkirchen)
No Assignees
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Vereins-IT/ezgg-lan-manager#11