0.3 - Levelsystem (#25) #92

edraft · 2022-11-06T22:29:23+01:00

edraft commented

2022-11-06 22:29:23 +01:00

#25

edraft added 11 commits 2022-11-06 22:29:23 +01:00

Added level module and migration #25 e1dbab3f4f

Added level model #25 5a3eb57c0b

Added level repo #25 e6e26b77f9

Added logic to handle default levels #25 666c527730

Improved logic to create roles for levels by default #25 861a847088

Improved logic to create roles for levels by default #25 8118d4edc3

Added level service #25 8aee72856c

Added logic to handle level by xp #25 164671a07c

Fixed level assignment #25 6e39154c75

Added logic to send message after level up #25 6640d70440

Fixed my config #25 23fde793db

edraft self-assigned this 2022-11-06 22:29:27 +01:00

edraft added this to the Sprint 0.3 project 2022-11-06 22:29:29 +01:00

edraft added this to the Sprint 0.3 milestone 2022-11-06 22:29:31 +01:00

edraft requested review from Ebola-Chan 2022-11-06 22:29:35 +01:00

edraft added 1 commit 2022-11-06 22:30:59 +01:00

"Repaired" workspace file #25 dc90258a28

edraft added 1 commit 2022-11-06 22:40:33 +01:00

Build files #25 1b587b049a

edraft added 1 commit 2022-11-06 22:43:44 +01:00

Added translation to level service #25 3d27295b14

edraft added 1 commit 2022-11-06 22:46:17 +01:00

Updated version of level project #25 63000636a8

Ebola-Chan requested changes 2022-11-07 21:53:55 +01:00

kdb-bot/src/bot_data/model/level.py

						
				@ -0,0 +112,4 @@

				        """)

				    @property

				    def delete_string(self) -> str:

Ebola-Chan commented

2022-11-07 21:21:33 +01:00

Evtl noch ein

def delete_by_server_id_String(self) -> str:
	return str(f"""
    	DELETE FROM `Levels`
        WHERE ServerId = {self.server._server_id}
    """)

und

def delete_string(self) -> str:

zu

def delete_by_id_string(self) -> str:

unbenennen?

Da wäre noch die Frage wegen der Konsistenz: Wollen wir wie bei den Get-Strings nur die ID übergeben oder "self"?

Sollte der Bot von z.B. einem Server genommen werden, können die Level zu den Server schnell und einfach gepurged werden.
Evtl kann dies auch zu einem kleinen Ticket werden

Evtl noch ein ```python def delete_by_server_id_String(self) -> str: return str(f""" DELETE FROM `Levels` WHERE ServerId = {self.server._server_id} """) ``` und ```python def delete_string(self) -> str: ``` zu ```python def delete_by_id_string(self) -> str: ``` unbenennen? Da wäre noch die Frage wegen der Konsistenz: Wollen wir wie bei den Get-Strings nur die ID übergeben oder "self"? Sollte der Bot von z.B. einem Server genommen werden, können die Level zu den Server schnell und einfach gepurged werden. Evtl kann dies auch zu einem kleinen Ticket werden

edraft commented

2022-11-07 22:13:52 +01:00

Aktuell wollen wir den Fall nicht abdecken, dass der Bot einen Server verlässt bzw. ignorieren dass. So kann der Bot zukünftig wiedergeholt werden und die "Konfig" ist dann nicht weg.

Daher halte ich den vorschlag nicht für brauchbar. Des weiteren wäre mein Ansatz an stelle das ganze aus Python zu machen (wegen Logging und so).

Aktuell wollen wir den Fall nicht abdecken, dass der Bot einen Server verlässt bzw. ignorieren dass. So kann der Bot zukünftig wiedergeholt werden und die "Konfig" ist dann nicht weg. Daher halte ich den vorschlag nicht für brauchbar. Des weiteren wäre mein Ansatz an stelle das ganze aus Python zu machen (wegen Logging und so).

👍 1

Ebola-Chan marked this conversation as resolved

Ebola-Chan approved these changes 2022-11-07 22:15:18 +01:00

edraft merged commit e27d2ae682 into 0.3

2022-11-07 22:15:37 +01:00

edraft deleted branch #25

2022-11-07 22:15:38 +01:00

edraft referenced this issue from a commit

2022-11-07 22:15:38 +01:00

Merge pull request '0.3 - Levelsystem (#25)' (#92) from #25 into 0.3

edraft removed this from the Sprint 0.3 milestone 2023-02-13 10:27:52 +01:00

Sign in to join this conversation.

No reviewers

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#92