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

0.3 - /level edit (#103) #108

Merged
edraft merged 5 commits from #103 into 0.3 2022-11-11 22:02:27 +01:00
Conversation 3 Commits 5 Files Changed 2 +105 -26
edraft commented 2022-11-10 16:34:33 +01:00
Owner
Copy Link

#103

Habe die Parameter ein wenig überarbeitet und daher keinen Dialog gewählt.

#103 Habe die Parameter ein wenig überarbeitet und daher keinen Dialog gewählt.
edraft self-assigned this 2022-11-10 16:34:33 +01:00
edraft added 1 commit 2022-11-10 16:34:33 +01:00
Added level edit command #103 44f22e6aee
edraft requested review from Ebola-Chan 2022-11-10 16:34:36 +01:00
edraft added this to the Sprint 0.3 project 2022-11-10 16:34:38 +01:00
edraft added this to the Sprint 0.3 milestone 2022-11-10 16:34:41 +01:00
edraft added 1 commit 2022-11-10 21:09:54 +01:00
Merge branch '0.3' into #103 50fbc3eb8c
edraft added 1 commit 2022-11-10 21:10:51 +01:00
Added is admin check to level edit #103 c70149e9c8
Ebola-Chan approved these changes 2022-11-11 01:42:48 +01:00
Ebola-Chan left a comment
Contributor
Copy Link

Sollte die Benachrichtogung für Anwender aktuell nicht wichtig sein oder wird anders gehandelt, können die Punkte ignoriert werden.

Sollte die Benachrichtogung für Anwender aktuell nicht wichtig sein oder wird anders gehandelt, können die Punkte ignoriert werden.
kdb-bot/src/modules/level/command/level_group.py Outdated
@@ -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)
Ebola-Chan commented 2022-11-11 01:33:50 +01:00
Contributor
Copy Link

Wenn die Farbe nicht geparsed werden kann, wäre es gut, wenn der Anwender darüber benachrichtigt wird.

Wenn die Farbe nicht geparsed werden kann, wäre es gut, wenn der Anwender darüber benachrichtigt wird.
edraft marked this conversation as resolved
kdb-bot/src/modules/level/command/level_group.py
@@ -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)
Ebola-Chan commented 2022-11-11 01:36:58 +01:00
Contributor
Copy Link

Auch hier, wenn die Permission nicht geparsed werden kann, soll der Anwender darüber benachrichtigt werden.

Auch hier, wenn die Permission nicht geparsed werden kann, soll der Anwender darüber benachrichtigt werden.
edraft marked this conversation as resolved
edraft commented 2022-11-11 07:28:18 +01:00
Author
Owner
Copy Link

Sollte die Benachrichtogung für Anwender aktuell nicht wichtig sein oder wird anders gehandelt, können die Punkte ignoriert 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.

> Sollte die Benachrichtogung für Anwender aktuell nicht wichtig sein oder wird anders gehandelt, können die Punkte ignoriert 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.
edraft added 1 commit 2022-11-11 07:28:22 +01:00
Merge branch '0.3' into #103 8fafc94118
edraft added 1 commit 2022-11-11 07:33:27 +01:00
Added error notification to level edit #103 b2087042bc
edraft requested review from Ebola-Chan 2022-11-11 07:33:34 +01:00
Ebola-Chan approved these changes 2022-11-11 21:59:04 +01:00
edraft merged commit e098416a29 into 0.3 2022-11-11 22:02:27 +01:00
edraft deleted branch #103 2022-11-11 22:02:27 +01:00
edraft removed this from the Sprint 0.3 milestone 2023-02-13 10:28:47 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Ebola-Chan
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 MonkeyDGoku1996 pvtkeks tammo
No Assignees edraft
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#108
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?