Rollen per Reaktion hinzufügen (#54) #58
No reviewers
Labels
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
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: sh-edraft.de/sh_discord_bot#58
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "#54"
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?
#54
Rollen per Reaktion hinzufügen (#54)to WIP: Rollen per Reaktion hinzufügen (#54)WIP: Rollen per Reaktion hinzufügen (#54)to Rollen per Reaktion hinzufügen (#54)fine by me
@ -56,0 +63,4 @@
"add": {
"success": "auto-role für die Nachricht {} wurde hinzugefügt :D",
"error": {
"not_found": "Nachricht {} nicht in {} nicht gefunden!",
Hier wurde "nicht" doppeltgemoppelt.
Schreibe "Nachricht {} wurde nicht in {} gefunden!"
fixed
@ -0,0 +72,4 @@
return str(f"""
UPDATE `AutoRoles`
SET `ServerId` = {self._server_id},
SET `DiscordMessageId` = {self._discord_message_id},
2 x SET in einem SQL-Statement?
@ -0,0 +72,4 @@
UPDATE `AutoRoleRules`
SET `AutoRoleId` = {self._auto_role_id},
SET `DiscordEmojiName` = {self._discord_emoji_name},
SET `DiscordRoleId` = {self._discord_role_id},
Mehrere SETs in einem SQL-Statement
@ -0,0 +8,4 @@
from bot_data.model.auto_role import AutoRole
from bot_data.model.auto_role_rule import AutoRoleRule
Allgemine Anmerkung ohne dass ich diese selbst geprüft habe: Können diese Methoden nur von uns ausgeführt werden?
Wenn Moderatoren diese Methoden auch ausführen können, müssen wir sicher gehen, dass diese keine Rollen/Regeln von anderen Servern aufgerufen werden können.
Kann man streichen, da die Methoden von Stellen aufgerufen werden, welche das schon prüfen.
@ -0,0 +49,4 @@
if result is None or len(result) == 0:
return None
result = result[0]
Wird das hier nicht zu einem Problem führen?
@ -0,0 +59,4 @@
id=result[0]
)
def get_auto_roles_by_server_id(self, id: int) -> AutoRole:
Soll hier nicht eine Liste der auto-roles zurückgegeben werden, wegen dem "roles" im Namen?
@ -0,0 +70,4 @@
id=result[0]
)
def find_auto_roles_by_server_id(self, id: int) -> Optional[AutoRole]:
Hier auch wieder eine Liste ausgeben wegen dem "roles" im Namen.
@ -0,0 +86,4 @@
id=result[0]
)
def get_auto_roles_by_message_id(self, id: int) -> AutoRole:
Methode umbenennen in "get_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.
@ -0,0 +97,4 @@
id=result[0]
)
def find_auto_roles_by_message_id(self, id: int) -> Optional[AutoRole]:
Methode umbenennen in "find_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.
@ -0,0 +103,4 @@
if result is None or len(result) == 0:
return None
result = result[0]
Führt das hier nicht zu einem Problem?