Skip to content
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

Convert !commands to slash commands #131

Open
nickserv opened this issue Dec 17, 2021 · 8 comments · May be fixed by #148
Open

Convert !commands to slash commands #131

nickserv opened this issue Dec 17, 2021 · 8 comments · May be fixed by #148
Assignees
Labels
enhancement New feature or request

Comments

@nickserv
Copy link
Member

nickserv commented Dec 17, 2021

Inspired by #46

Application commands provide a better UX for running bot commands in Discord (fixing common issues of ours like moderators having to delete commands that users accidentally run) while also allowing for more relaxed bot permissions.

Once we upgrade to discord.js 13, we can use their registering slash commands example.

@nickserv nickserv added the enhancement New feature or request label Dec 17, 2021
@vcarl
Copy link
Member

vcarl commented Dec 17, 2021

This seems like one benefit we'd get by updating discord.js, maybe that's a better title?

@nickserv
Copy link
Member Author

nickserv commented Dec 18, 2021

I'm intentionally keeping this as a separate issue, since this is more of a UX improvement and the handler format is not required in discord.js 13. Neither PR has this yet, and I might move the refactor to another PR anyway to make it easier to review.

@nickserv

This comment has been minimized.

@vcarl

This comment has been minimized.

@vcarl vcarl changed the title Enable application commands Convert !commands to application/slash commands Dec 20, 2021
@nickserv nickserv self-assigned this Dec 30, 2021
@nickserv
Copy link
Member Author

nickserv commented Jan 2, 2022

Renaming to specifically use slash commands, since we don't seem to need the other application command types (user and message context menu commands) yet. Slash commands are specifically used when typing messages, which is most similar to how we're currently using the bot.

@nickserv nickserv changed the title Convert !commands to application/slash commands Convert !commands to slash commands Jan 2, 2022
@nickserv nickserv linked a pull request Jan 2, 2022 that will close this issue
@Padmanabh82
Copy link

Ye, we need this

@nickserv
Copy link
Member Author

What's the status of this? I noticed another PR got merged, but I'm not seeing our slash commands on Discord.

@vcarl
Copy link
Member

vcarl commented May 27, 2022

I haven't made any changes to affect this, but I did implement command deployment in #198 which should facilitate this. Our only command atm is the anonymous report

const cmds: CommandConfig[] = [report];

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants