Ebola-Chan
  • Joined on 2022-07-20
Ebola-Chan approved sh-edraft.de/sh_discord_bot#361 2023-09-24 20:31:53 +02:00
#360
Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#361 2023-09-24 20:31:53 +02:00
#360

in der Migration definierst du die Variable als BIGINT aber in der Weboberfläche als String?

Ebola-Chan suggested changes for sh-edraft.de/sh_discord_bot#357 2023-09-10 14:45:17 +02:00
Added check with -xp to get_level
Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#357 2023-09-10 14:45:17 +02:00
Added check with -xp to get_level

Ich meine hier fehlt ein Return.

Ebola-Chan approved sh-edraft.de/sh_discord_bot#356 2023-09-10 14:21:25 +02:00
support in master

Einen Änderungsvorschlag habe ich, aber musst du wissen, ob es sich lohnt diesen umzusetzen.

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#356 2023-09-10 14:21:25 +02:00
support in master

Ich sehe sehr oft doppelten Code. Wie wäre es, wenn wir dies in eine Methode auslagern?

Ebola-Chan approved sh-edraft.de/sh_discord_bot#355 2023-08-24 23:54:04 +02:00
1.1.1
Ebola-Chan approved sh-edraft.de/sh_discord_bot#352 2023-08-23 20:27:31 +02:00
1.1.0
Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Hier fehlt wieder ein continue

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Die Fehlermeldung "Cannot get user" ist für mich zu ungenau. Was ist wenn es zu einem Fehler kommt, wenn der Bot versucht den User zu speichern?

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Wie unten Angemerkt kommt es zu einer NullReferenceException wenn user None ist. Evtl nach Protokollieren ein continue?

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Fehlermeldung auch hier zu allgemein.

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Auch hier fehlt ein continue

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Hier sollte es zu einer NullReferenceException kommen, wenn user None ist.

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Hier sollte es zu einer NullReferenceException kommen, wenn user None ist.

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

An anderen Stellen wird geprüft, ob der Eintrag auch wirklich gespeichert wurde.

Ebola-Chan commented on pull request sh-edraft.de/sh_discord_bot#321 2023-06-20 23:32:32 +02:00
#292_shutdown_procedure

Wie unten Angemerkt kommt es zu einer NullReferenceException wenn user None ist. Evtl nach Protokollieren ein continue?