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

Added user set command #22 #143

Merged
edraft merged 6 commits from #22 into 0.3 2022-12-23 15:48:38 +01:00
Conversation 4 Commits 6 Files Changed 2 +107 -39
Ebola-Chan commented 2022-12-17 19:26:40 +01:00
Contributor
Copy Link

#22

#22
Ebola-Chan added this to the Sprint 0.3 milestone 2022-12-17 19:26:40 +01:00
Ebola-Chan self-assigned this 2022-12-17 19:26:40 +01:00
Ebola-Chan added 3 commits 2022-12-17 19:26:41 +01:00
Added user set command ec30069ff5
Added translation for user set command cf610b770b
Fixed not checking level after new XP is assinged to user #22 5a4c2901f5
Ebola-Chan requested review from edraft 2022-12-17 19:26:45 +01:00
Ebola-Chan added this to the Sprint 0.3 project 2022-12-17 19:26:53 +01:00
Ebola-Chan added a new dependency 2022-12-17 21:54:01 +01:00
#144 Added user remove command #23
edraft added 1 commit 2022-12-18 11:02:09 +01:00
Merge branch '0.3' into #22 5447d502cc
edraft requested changes 2022-12-18 11:05:37 +01:00
kdb-bot/src/modules/base/command/user_group.py Outdated
@@ -170,0 +199,4 @@
self._db.save_changes()
await self._level.check_level(member)
except Exception as e:
self._logger.error(__name__, f'Value couldn\'t be converted to int', e)
edraft commented 2022-12-18 11:05:23 +01:00
Owner
Copy Link

Relativ ungenau. Entweder änderst du die beschreibung von dem error oder du machst zwei try except blöcke, einmal für den type cast und dann für die manipulation des users inkl. level check

Relativ ungenau. Entweder änderst du die beschreibung von dem error oder du machst zwei try except blöcke, einmal für den type cast und dann für die manipulation des users inkl. level check
Ebola-Chan commented 2022-12-19 00:19:09 +01:00
Author
Contributor
Copy Link

Das Exceptionhandling für das "check_level" würde ich lieber dann in der Methode selber machen und das in einem separaten Issue, da ich dafür weitere Exceptionklassen erstellen möchte, um Exceptions in der Methode besser von den allgemeinen Databaseexceptions zu differenzieren.

Das Exceptionhandling für das "check_level" würde ich lieber dann in der Methode selber machen und das in einem separaten Issue, da ich dafür weitere Exceptionklassen erstellen möchte, um Exceptions in der Methode besser von den allgemeinen Databaseexceptions zu differenzieren.
Ebola-Chan marked this conversation as resolved
edraft requested changes 2022-12-18 11:07:29 +01:00
kdb-bot/src/modules/base/command/user_group.py
@@ -170,0 +215,4 @@
@set.autocomplete('atr')
async def set_autocomplete(self, interaction: discord.Interaction, current: str) -> List[app_commands.Choice[str]]:
atr_list = ['xp']
edraft commented 2022-12-18 11:07:27 +01:00
Owner
Copy Link

Es wäre ne idee, dass diese strings irgendwie aus translation oder config kommen. Also das man da halt ne liste von diesen Strings hat. Ist auch diesen drei Befehlen immer der Fall

Es wäre ne idee, dass diese strings irgendwie aus translation oder config kommen. Also das man da halt ne liste von diesen Strings hat. Ist auch diesen drei Befehlen immer der Fall
Ebola-Chan marked this conversation as resolved
Ebola-Chan added 1 commit 2022-12-18 22:44:37 +01:00
Improved error handling and changed attribute names from constants to variables which are located in the language file #22 1b9553e63b
edraft requested changes 2022-12-19 07:59:41 +01:00
kdb-bot/src/modules/base/command/user_group.py Outdated
@@ -145,3 +160,3 @@
match atr:
case 'xp':
case self._atr_class.xp:
edraft commented 2022-12-19 07:59:37 +01:00
Owner
Copy Link

Mach das bitte mit nem elif und als dict in der Klasse und keine weitere Klasse dafür einführen
https://stackoverflow.com/questions/72638083/python-match-case-dictionary-keys

Mach das bitte mit nem elif und als dict in der Klasse und keine weitere Klasse dafür einführen https://stackoverflow.com/questions/72638083/python-match-case-dictionary-keys
Ebola-Chan commented 2022-12-22 20:55:57 +01:00
Author
Contributor
Copy Link

Match-Case-Statement gegen If-Else-Statement getauscht

Match-Case-Statement gegen If-Else-Statement getauscht
Ebola-Chan marked this conversation as resolved
kdb-bot/src/modules/base/command/user_group.py Outdated
@@ -169,0 +211,4 @@
self._logger.error(__name__, f'String value couldn\'t be converted to int', te)
await self._message_service.send_interaction_msg(ctx.interaction, self._t.transform('modules.base.user.set.error.type_error'))
return
except DatabaseError as de:
edraft commented 2022-12-19 07:53:43 +01:00
Owner
Copy Link

Der ist bei weitem unnötig xD
Wenn wir mal in check_level reinschauen sehen wir, dass dort alle Fehler abefangen werden.

Des Weiteren ist DatabaseError um das update_user abzufangen ungenau bzw als solches auch nicht ötig.

Der ist bei weitem unnötig xD Wenn wir mal in check_level reinschauen sehen wir, dass dort alle Fehler abefangen werden. Des Weiteren ist DatabaseError um das update_user abzufangen ungenau bzw als solches auch nicht ötig.
Ebola-Chan commented 2022-12-22 20:56:43 +01:00
Author
Contributor
Copy Link

Errorhandling entfernt. Nur TypeError wird behandelt.

Errorhandling entfernt. Nur TypeError wird behandelt.
Ebola-Chan marked this conversation as resolved
kdb-bot/src/modules/base/command/user_group.py Outdated
@@ -169,0 +216,4 @@
await self._message_service.send_interaction_msg(ctx.interaction, self._t.transform('modules.base.user.set.error.database_error'))
return
except Exception as e:
self._logger.error(__name__, f'An error occurred while setting xp', e)
edraft commented 2022-12-19 07:54:36 +01:00
Owner
Copy Link

DieseBlock ist wegen command error handling an und für auch nicht nötig. Hier könnte man wenn man wollte die Datenbankfehler abfangen

DieseBlock ist wegen command error handling an und für auch nicht nötig. Hier könnte man wenn man wollte die Datenbankfehler abfangen
Ebola-Chan marked this conversation as resolved
Ebola-Chan added 1 commit 2022-12-22 20:39:28 +01:00
Removed error handling for database errors and replaced match-case-statements with if-else-statements #22 40e53de3f2
edraft approved these changes 2022-12-23 15:48:26 +01:00
edraft merged commit 6c6169f7ee into 0.3 2022-12-23 15:48:38 +01:00
edraft deleted branch #22 2022-12-23 15:48:38 +01:00
edraft removed this from the Sprint 0.3 milestone 2023-02-13 10:29:40 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
edraft
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 0.3
Assignees
Clear assignees
ci edraft edraft-dev
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Blocks
#144 Added user remove command #23
sh-edraft.de/sh_discord_bot
Reference: sh-edraft.de/sh_discord_bot#143
No description provided.
Delete Branch "#22"

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?