0.3 - /level edit (#103) #108
No reviewers
Labels
No Label
MVP
web-interface
bug
bugfix
cli
discussion
documentation
duplicate
expense
high
expense
low
expense
medium
expense
very-high
help wanted
improvement
invalid
priority
blocker
priority
high
priority
low
priority
medium
question
refactoring
release
story
wontfix
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: sh-edraft.de/sh_discord_bot#108
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "#103"
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?
#103
Habe die Parameter ein wenig überarbeitet und daher keinen Dialog gewählt.
Sollte die Benachrichtogung für Anwender aktuell nicht wichtig sein oder wird anders gehandelt, können die Punkte ignoriert werden.
@ -198,0 +229,4 @@
try:
level_from_db.color = hex(discord.Colour.from_str(color).value)
except Exception as e:
self._logger.error(__name__, f'Error parsing color {color}', e)
Wenn die Farbe nicht geparsed werden kann, wäre es gut, wenn der Anwender darüber benachrichtigt wird.
@ -198,0 +239,4 @@
try:
level_from_db.permissions = discord.Permissions(permissions).value
except Exception as e:
self._logger.error(__name__, f'Error parsing permissions {permissions}', e)
Auch hier, wenn die Permission nicht geparsed werden kann, soll der Anwender darüber benachrichtigt werden.
In den Fällen gebe ich dir recht, hier hat der Nutzer keine andere möglichkeit darauf zu schließen das etwas schief gelaufen ist.