Port of cmds to slashcommands #294
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If no twitch id / secret was provided the twitchnotifier init script would return a nil pointer. The timer would then try to call the Handle function of this (nil) pointer.
If no twitch id / secret was provided the twitchnotifier init script would return a nil pointer. The timer would then try to call the Handle function of this (nil) pointer.
Discord showed a message, that the ban interaction failed, although it successfully banned. Add Defer statement to show that a response will later be available
Add the slashcommand port of the bug command.
Add Clear slash command
Clear command was deleting 1 message less than expected.
Interactions were wrongly marked as failed, because no follup message was sent.
|
Sadly, I already ported the ban command and merged it with the kick and report command into one single slash command, so I guess, I will remove it and keep the other changes. Thank you really much for your effort though. The next commands I want to port are the following:
Just to avoid the possibility of further collisions. ^^ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The commands
are ported to the new slashcommand system.
It is pretty much a 1-to-1 port. (E.g. most code is copied from the old versions).
Also fixed a bug, that the bot would crash if no twitchnotifier were used.
part of #290