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

1.0.0.rc1 #244

Merged
edraft merged 14 commits from 1.0.0.rc1 into 1.0.0 2023-03-11 01:12:31 +01:00
Conversation 2 Commits 14 Files Changed 102 +4
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 1664c67763 - Show all commits

2
kdb-bot/src/bot_data/model/user_joined_game_server.py
View File

@@ -41,6 +41,8 @@ class UserJoinedGameServer(TableABC):
@property
def time(self) -> float:
if self._leaved_on is None or self._joined_on is None:
return 0
return round((self.leaved_on - self.joined_on).total_seconds() / 3600, 2)
edraft marked this conversation as resolved
Ebola-Chan commented 2023-03-09 23:32:02 +01:00
Review
Copy Link

Folgende Frage: Wenn ein User innerhalb einer Sekunde joined und leaved, kann hier nicht durch 0 geteilt werden? Es könnte sein, dass durch die Millisekunden automatisch 1 Sekunde durch total_seconds() ausgegeben wird.

Folgende Frage: Wenn ein User innerhalb einer Sekunde joined und leaved, kann hier nicht durch 0 geteilt werden? Es könnte sein, dass durch die Millisekunden automatisch 1 Sekunde durch ```total_seconds()``` ausgegeben wird.
edraft commented 2023-03-10 11:16:19 +01:00
Review
Copy Link

Passt:

round((datetime.strptime("2023-03-09 19:37:30.170896", "%Y-%m-%d %H:%M:%S.%f") - datetime.strptime("2023-03-09 19:37:30.170896", "%Y-%m-%d %H:%M:%S.%f")).total_seconds() / 3600, 2) = 0.0
Passt: ```py round((datetime.strptime("2023-03-09 19:37:30.170896", "%Y-%m-%d %H:%M:%S.%f") - datetime.strptime("2023-03-09 19:37:30.170896", "%Y-%m-%d %H:%M:%S.%f")).total_seconds() / 3600, 2) = 0.0 ```
@property

2
kdb-bot/src/bot_data/model/user_joined_voice_channel.py
View File

@@ -47,6 +47,8 @@ class UserJoinedVoiceChannel(TableABC):
@property
def time(self) -> float:
if self._leaved_on is None or self._joined_on is None:
return 0
return round((self.leaved_on - self.joined_on).total_seconds() / 3600, 2)
edraft marked this conversation as resolved
Ebola-Chan commented 2023-03-09 23:32:45 +01:00
Review
Copy Link

Hier auch die selbe Frage mit durch 0 dividieren.

Hier auch die selbe Frage mit durch 0 dividieren.
@property