Rollen per Reaktion hinzufügen (#54) #58

Merged
edraft merged 20 commits from #54 into master 2022-10-05 19:08:22 +02:00
Owner

#54

#54
edraft added 15 commits 2022-10-04 22:32:49 +02:00
edraft requested review from Ebola-Chan 2022-10-04 22:33:38 +02:00
edraft requested review from edraft-dev 2022-10-04 22:33:38 +02:00
Ebola-Chan was assigned by edraft 2022-10-04 22:33:43 +02:00
edraft-dev was assigned by edraft 2022-10-04 22:33:43 +02:00
edraft added this to the Sprint 0.2 project 2022-10-04 22:33:46 +02:00
edraft added this to the Sprint 0.1 milestone 2022-10-04 22:33:48 +02:00
edraft modified the milestone from Sprint 0.1 to Sprint 0.2 2022-10-04 22:33:50 +02:00
edraft-dev changed title from Rollen per Reaktion hinzufügen (#54) to WIP: Rollen per Reaktion hinzufügen (#54) 2022-10-05 16:42:34 +02:00
edraft added 1 commit 2022-10-05 16:43:28 +02:00
edraft-dev changed title from WIP: Rollen per Reaktion hinzufügen (#54) to Rollen per Reaktion hinzufügen (#54) 2022-10-05 16:43:32 +02:00
edraft added 1 commit 2022-10-05 16:52:13 +02:00
edraft-dev approved these changes 2022-10-05 16:54:32 +02:00
edraft-dev left a comment
Owner

fine by me

fine by me
Ebola-Chan reviewed 2022-10-05 17:35:38 +02:00
@ -56,0 +63,4 @@
"add": {
"success": "auto-role für die Nachricht {} wurde hinzugefügt :D",
"error": {
"not_found": "Nachricht {} nicht in {} nicht gefunden!",
Contributor

Hier wurde "nicht" doppeltgemoppelt.
Schreibe "Nachricht {} wurde nicht in {} gefunden!"

Hier wurde "nicht" doppeltgemoppelt. Schreibe "Nachricht {} wurde nicht in {} gefunden!"
Author
Owner

fixed

fixed
edraft marked this conversation as resolved
edraft added 1 commit 2022-10-05 17:38:24 +02:00
edraft added 1 commit 2022-10-05 18:09:30 +02:00
Ebola-Chan requested changes 2022-10-05 18:53:07 +02:00
@ -0,0 +72,4 @@
return str(f"""
UPDATE `AutoRoles`
SET `ServerId` = {self._server_id},
SET `DiscordMessageId` = {self._discord_message_id},
Contributor

2 x SET in einem SQL-Statement?

2 x SET in einem SQL-Statement?
edraft marked this conversation as resolved
@ -0,0 +72,4 @@
UPDATE `AutoRoleRules`
SET `AutoRoleId` = {self._auto_role_id},
SET `DiscordEmojiName` = {self._discord_emoji_name},
SET `DiscordRoleId` = {self._discord_role_id},
Contributor

Mehrere SETs in einem SQL-Statement

Mehrere SETs in einem SQL-Statement
edraft marked this conversation as resolved
@ -0,0 +8,4 @@
from bot_data.model.auto_role import AutoRole
from bot_data.model.auto_role_rule import AutoRoleRule
Contributor

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.

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.
Contributor

Kann man streichen, da die Methoden von Stellen aufgerufen werden, welche das schon prüfen.

Kann man streichen, da die Methoden von Stellen aufgerufen werden, welche das schon prüfen.
edraft marked this conversation as resolved
@ -0,0 +49,4 @@
if result is None or len(result) == 0:
return None
result = result[0]
Contributor

Wird das hier nicht zu einem Problem führen?

Wird das hier nicht zu einem Problem führen?
edraft marked this conversation as resolved
@ -0,0 +59,4 @@
id=result[0]
)
def get_auto_roles_by_server_id(self, id: int) -> AutoRole:
Contributor

Soll hier nicht eine Liste der auto-roles zurückgegeben werden, wegen dem "roles" im Namen?

Soll hier nicht eine Liste der auto-roles zurückgegeben werden, wegen dem "roles" im Namen?
edraft marked this conversation as resolved
@ -0,0 +70,4 @@
id=result[0]
)
def find_auto_roles_by_server_id(self, id: int) -> Optional[AutoRole]:
Contributor

Hier auch wieder eine Liste ausgeben wegen dem "roles" im Namen.

Hier auch wieder eine Liste ausgeben wegen dem "roles" im Namen.
edraft marked this conversation as resolved
@ -0,0 +86,4 @@
id=result[0]
)
def get_auto_roles_by_message_id(self, id: int) -> AutoRole:
Contributor

Methode umbenennen in "get_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.

Methode umbenennen in "get_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.
edraft marked this conversation as resolved
@ -0,0 +97,4 @@
id=result[0]
)
def find_auto_roles_by_message_id(self, id: int) -> Optional[AutoRole]:
Contributor

Methode umbenennen in "find_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.

Methode umbenennen in "find_auto_role_by_message_id", da es hier zu keiner Liste kommen sollte.
edraft marked this conversation as resolved
@ -0,0 +103,4 @@
if result is None or len(result) == 0:
return None
result = result[0]
Contributor

Führt das hier nicht zu einem Problem?

Führt das hier nicht zu einem Problem?
edraft marked this conversation as resolved
edraft added 1 commit 2022-10-05 19:05:01 +02:00
Ebola-Chan approved these changes 2022-10-05 19:08:00 +02:00
edraft merged commit 359a614a1a into master 2022-10-05 19:08:22 +02:00
edraft deleted branch #54 2022-10-05 19:08:22 +02:00
edraft removed this from the Sprint 0.2 milestone 2023-02-13 10:32:07 +01:00
Sign in to join this conversation.
No description provided.