-
-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove .commands #333
base: main
Are you sure you want to change the base?
Remove .commands #333
Conversation
we also might want to just remove the MessageAdd path for deleting responses. it would be a breaking change from nodefraction, but i think a pretty welcome one as messages being deleted with the X isn't actually documented anywhere and using the context menu is similar effort to reacting anyways. this would also save us a bit of time on processing each message, which is a plus |
Also added Manage Messages check to delete any message because why not |
Signed-off-by: TheKodeToad <[email protected]>
I'm using GitHub web editor 😔 Signed-off-by: TheKodeToad <[email protected]>
I think people use them too often to be removed |
Since this seemed to be what a few people wanted?