Rollen per Reaktion hinzufügen (#54) #58
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 AutoRolefrom bot_data.model.auto_role_rule import AutoRoleRuleAllgemine 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 Noneresult = 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 Noneresult = result[0]Führt das hier nicht zu einem Problem?