sh-edraft.de/sh_discord_bot
5
1
Fork 1
Code Issues 26 Pull Requests Actions Packages Projects 3 Releases 27 Wiki Activity

Verwarnungssystem #35 #235

Merged
edraft merged 7 commits from #35 into 1.0.0 2023-02-24 08:49:09 +01:00
Conversation 2 Commits 7 Files Changed 13 +507 -2
edraft commented 2023-02-22 08:44:54 +01:00
Owner
Copy Link

#35

#35
edraft self-assigned this 2023-02-22 08:44:54 +01:00
edraft added 5 commits 2023-02-22 08:44:55 +01:00
Added user warning model #35 89bcb655d2
Added user warning repository #35 8ff21debf0
Added warnings service #35 19f47a01e5
Improved warning handling #35 1eb625fe7a
Added warning commands #35 2674af64e9
edraft added this to the Sprint 1.0.0 project 2023-02-22 08:44:59 +01:00
edraft requested review from Ebola-Chan 2023-02-22 08:45:04 +01:00
Ebola-Chan requested changes 2023-02-22 22:26:33 +01:00
kdb-bot/src/bot_data/service/user_warnings_repository_service.py Outdated
@@ -0,0 +16,4 @@
logger: DatabaseLogger,
db_context: DatabaseContextABC,
users: UserRepositoryABC,
servers: ServerRepositoryABC,
Ebola-Chan commented 2023-02-22 21:19:25 +01:00
Contributor
Copy Link

Du übergibst servers: ServerRepositoryABC aber dies wird in der Definition von des Constructors nirgendswo zugewiesen.
Ich sehe auch nicht dass self._servers in der Klasse irgendwo verwendet wird. Somit könnte der Parameter weg.

Du übergibst ```servers: ServerRepositoryABC``` aber dies wird in der Definition von des Constructors nirgendswo zugewiesen. Ich sehe auch nicht dass ```self._servers``` in der Klasse irgendwo verwendet wird. Somit könnte der Parameter weg.
edraft marked this conversation as resolved
kdb-bot/src/modules/base/command/user_group.py Outdated
@@ -344,0 +371,4 @@
title=member.name, description=self._t.transform("modules.base.user.atr.warnings"), color=int("ef9d0d", 16)
)
warnings_id_string = ""
Ebola-Chan commented 2023-02-22 22:07:43 +01:00
Contributor
Copy Link

Evtl. die Zugriffe auf die Datenbank hier verringern, in dem das Ergebnis in einer Variable zwischengespeichert wird.

Von:

warnings_id_string = ""
for warning in self._user_warnings.get_user_warnings_by_user_id(user.id):
    warnings_id_string += f"{warning.id}\n"

warnings_description_string = ""
for warning in self._user_warnings.get_user_warnings_by_user_id(user.id):
    warnings_description_string += f"{warning.description}\n"

Zu:

warnings = self._user_warnings.get_user_warnings_by_user_id(user.id)

warnings_id_string = ""
for warning in warnings:
    warnings_id_string += f"{warning.id}\n"

warnings_description_string = ""
for warning in warnings:
    warnings_description_string += f"{warning.description}\n"

Auch wenn unwahrscheinlich, aber ich meine dass dadurch auch eine zwischenzeitliche Änderung in der Datenbank keine Auswirkung auf die Formatierung des Embeds hat.

Evtl. die Zugriffe auf die Datenbank hier verringern, in dem das Ergebnis in einer Variable zwischengespeichert wird. Von: ```python warnings_id_string = "" for warning in self._user_warnings.get_user_warnings_by_user_id(user.id): warnings_id_string += f"{warning.id}\n" warnings_description_string = "" for warning in self._user_warnings.get_user_warnings_by_user_id(user.id): warnings_description_string += f"{warning.description}\n" ``` Zu: ```python warnings = self._user_warnings.get_user_warnings_by_user_id(user.id) warnings_id_string = "" for warning in warnings: warnings_id_string += f"{warning.id}\n" warnings_description_string = "" for warning in warnings: warnings_description_string += f"{warning.description}\n" ``` Auch wenn unwahrscheinlich, aber ich meine dass dadurch auch eine zwischenzeitliche Änderung in der Datenbank keine Auswirkung auf die Formatierung des Embeds hat.
edraft marked this conversation as resolved
kdb-bot/src/modules/base/command/user_group.py
@@ -344,0 +399,4 @@
async def add(self, ctx: Context, member: discord.Member, description: str):
self._logger.debug(__name__, f"Received command user warning add {ctx}:{member},{description}")
try:
await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success"))
Ebola-Chan commented 2023-02-22 22:10:45 +01:00
Contributor
Copy Link

Hier wird zuerst eine Message gesendet bevor versucht wird eine Warnung hinzuzufügen.
Bitte die beiden Zeilen tauschen:

await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success"))
await self._user_warnings_service.add_warnings(member, description, ctx.author.id)
Hier wird zuerst eine Message gesendet bevor versucht wird eine Warnung hinzuzufügen. Bitte die beiden Zeilen tauschen: ```python await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.add.success")) await self._user_warnings_service.add_warnings(member, description, ctx.author.id) ```
edraft marked this conversation as resolved
kdb-bot/src/modules/base/command/user_group.py
@@ -344,0 +413,4 @@
async def remove(self, ctx: Context, warning_id: int):
self._logger.debug(__name__, f"Received command user warning remove {ctx}:{warning_id}")
try:
await self._message_service.send_ctx_msg(ctx, self._t.transform("modules.base.warnings.remove.success"))
Ebola-Chan commented 2023-02-22 22:11:45 +01:00
Contributor
Copy Link

Auch hier wird zuerst eine Nachricht versendet, bevor die eine Aktion stattfindet. Auch hier die beiden Zeilen tauschen.

Auch hier wird zuerst eine Nachricht versendet, bevor die eine Aktion stattfindet. Auch hier die beiden Zeilen tauschen.
edraft marked this conversation as resolved
edraft added 1 commit 2023-02-23 08:46:53 +01:00
Merge branch '1.0.0' into #35 e5fb9fd504
edraft added 1 commit 2023-02-23 08:58:08 +01:00
Code refactoring #35 2d995544c3
edraft requested review from Ebola-Chan 2023-02-23 08:58:47 +01:00
Ebola-Chan approved these changes 2023-02-23 23:35:19 +01:00
edraft merged commit a588f8bf72 into 1.0.0 2023-02-24 08:49:09 +01:00
edraft deleted branch #35 2023-02-24 08:49:09 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Ebola-Chan
Labels
Clear labels
MVP
web-interface
bug
Something is not working
bugfix
Solution of a problem
cli
Changes for command line interface
discussion
Something needs to be discussed
documentation
Indicates a need for improvements or additions to documentation
duplicate
This issue or pull request already exists
expense
high
expense
low
expense
medium
expense
very-high
help wanted
Need some help
improvement
Improvement of a feature
invalid
Something is wrong
priority
blocker
priority
high
priority
low
priority
medium
question
More information is needed
refactoring
Code need some refactoring
release
Create a release
story
New feature
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project Sprint 1.0.0
Assignees
Clear assignees
ci edraft edraft-dev MonkeyDGoku1996 pvtkeks tammo
No Assignees edraft
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#235
No description provided.
Delete Branch "#35"

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?