sh-edraft.de/sh_discord_bot
5
1
Fork 1
Code Issues26 Pull Requests Actions Packages Projects3 Releases27 Wiki Activity

0.3 - Levelsystem () #92

Merged
edraft merged 15 commits from #25 into 0.3 2022-11-07 22:15:37 +01:00
Conversation 0 Commits 15 Files Changed 85 +1005 -110
edraft commented 2022-11-06 22:29:23 +01:00
Owner

#25
edraft added 11 commits 2022-11-06 22:29:23 +01:00
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
edraft added 1 commit 2022-11-06 22:40:33 +01:00
edraft added 1 commit 2022-11-06 22:43:44 +01:00
edraft added 1 commit 2022-11-06 22:46:17 +01:00
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
Contributor

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
Author
Owner

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 removed this from the Sprint 0.3 milestone 2023-02-13 10:27:52 +01:00
Sign in to join this conversation.
No reviewers
Ebola-Chan
No Label
MVP
web-interface
bug
bugfix
cli
discussion
documentation
duplicate
expense
high
expense
low
expense
medium
expense
very-high
help wanted
improvement
invalid
priority
blocker
priority
high
priority
low
priority
medium
question
refactoring
release
story
wontfix
No Milestone
No project
Sprint 0.3
No Assignees
edraft
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
No description provided.
Delete Branch "#25"

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?