0.3 - level down (#28) #98

Merged
edraft merged 4 commits from #28 into 0.3 2022-11-09 10:03:22 +01:00
Owner

#28

#28
edraft added this to the Sprint 0.3 milestone 2022-11-08 17:50:37 +01:00
edraft self-assigned this 2022-11-08 17:50:37 +01:00
edraft added 2 commits 2022-11-08 17:50:38 +01:00
edraft added this to the Sprint 0.3 project 2022-11-08 17:50:42 +01:00
edraft requested review from Ebola-Chan 2022-11-08 17:50:45 +01:00
edraft force-pushed #28 from e5a343a08a to f69c1b19d6 2022-11-08 21:56:57 +01:00 Compare
edraft force-pushed #28 from f69c1b19d6 to faaadb0009 2022-11-08 22:01:21 +01:00 Compare
edraft added 1 commit 2022-11-08 22:04:23 +01:00
# Conflicts:
#	kdb-bot/src/bot/translation/de.json
#	kdb-bot/src/modules/level/command/level_group.py
edraft changed title from WIP: 0.3 - level down (#28) to 0.3 - level down (#28) 2022-11-08 22:04:27 +01:00
Ebola-Chan requested changes 2022-11-09 01:15:38 +01:00
@ -241,0 +248,4 @@
server = self._servers.get_server_by_discord_id(ctx.guild.id)
user = self._users.get_user_by_discord_id_and_server_id(member.id, server.server_id)
level = self._level_service.get_level(user)
Contributor

Ich würde "level" in "user_level" unbenennen, damit man beim lesen sofort sieht, dass es sich hier nicht um irgendein Level handelt.
Ist aber nicht schlimm wenn es so bleibt.

Ich würde "level" in "user_level" unbenennen, damit man beim lesen sofort sieht, dass es sich hier nicht um irgendein Level handelt. Ist aber nicht schlimm wenn es so bleibt.
Contributor

Ich muss hier auch notieren, dass auch andere Stellen exestieren, bei dem ein Level eines Users in eine Variable geladen wird, welcher nur "level" heißt.

Ich muss hier auch notieren, dass auch andere Stellen exestieren, bei dem ein Level eines Users in eine Variable geladen wird, welcher nur "level" heißt.
Author
Owner

Hast recht, aber aufgrund der anderen stellen würde ich das vorerst so lassen

Hast recht, aber aufgrund der anderen stellen würde ich das vorerst so lassen
Ebola-Chan marked this conversation as resolved
@ -241,0 +262,4 @@
self._users.update_user(user)
self._db.save_changes()
await self._message_service.send_ctx_msg(ctx, self._t.transform('modules.level.down.success').format(member.name, new_level.name))
await self._level_service.set_level(user)
Contributor

Evtl die Aufrufe

await self._message_service.send_ctx_msg(ctx, self._t.transform('modules.level.down.success').format(member.name, new_level.name))
await self._level_service.set_level(user)

umtauschen

await self._level_service.set_level(user)
await self._message_service.send_ctx_msg(ctx, self._t.transform('modules.level.down.success').format(member.name, new_level.name))

Somit wird die Erfolgmeldung erst auch nach dem Setzen der Rolle ausgegeben.

Evtl die Aufrufe ```python await self._message_service.send_ctx_msg(ctx, self._t.transform('modules.level.down.success').format(member.name, new_level.name)) await self._level_service.set_level(user) ``` umtauschen ```python await self._level_service.set_level(user) await self._message_service.send_ctx_msg(ctx, self._t.transform('modules.level.down.success').format(member.name, new_level.name)) ``` Somit wird die Erfolgmeldung erst auch nach dem Setzen der Rolle ausgegeben.
Author
Owner

Nope :D

Die Erfolgsmeldung kommt, wenn der Eintrag in der Datenbank ohne Fehler gespeichert werden konnte. Im Zweifelsfall reicht uns das immer.
Die Funktion set_level schickt ebenfalls nochmal ne eigene Nachricht, siehe aufm Testserver in Kanal bot_commands

Nope :D Die Erfolgsmeldung kommt, wenn der Eintrag in der Datenbank ohne Fehler gespeichert werden konnte. Im Zweifelsfall reicht uns das immer. Die Funktion set_level schickt ebenfalls nochmal ne eigene Nachricht, siehe aufm Testserver in Kanal bot_commands
Ebola-Chan marked this conversation as resolved
edraft added 1 commit 2022-11-09 08:51:11 +01:00
edraft requested review from Ebola-Chan 2022-11-09 08:51:15 +01:00
Ebola-Chan approved these changes 2022-11-09 09:58:45 +01:00
edraft merged commit 054266bfa0 into 0.3 2022-11-09 10:03:22 +01:00
edraft deleted branch #28 2022-11-09 10:03:22 +01:00
edraft removed this from the Sprint 0.3 milestone 2023-02-13 10:28:03 +01:00
Sign in to join this conversation.
No description provided.