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

Implement application commands #6

Closed
teotwaki opened this issue Sep 30, 2021 · 0 comments · Fixed by #11
Closed

Implement application commands #6

teotwaki opened this issue Sep 30, 2021 · 0 comments · Fixed by #11
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@teotwaki
Copy link
Owner

It doesn't look like Discord is changing its mind when it comes to application commands, so they probably need to be implemented at some point. This is the task I'm currently working on.

I've tried playing around with Poise, but I wasn't a fan of how opinionated it is. That's not meant as a diss or anything, it's just that I'm opinionated as well, and our opinions don't really match. I'm sure that if I had started off building liro using Poise, I would be very happy with it.

I'm taking the overly verbose and declarative route that is currently available in Serenity. I'll see if I get around to publishing a branch with my current WIP, although it may not be super interesting. The main part that I'm trying to figure out is restructuring the existing commands and making that mesh with the application commands, so that I don't need to have two parallel implementations of each command.

@teotwaki teotwaki added the enhancement New feature or request label Sep 30, 2021
@teotwaki teotwaki added this to the 1.0 milestone Sep 30, 2021
@teotwaki teotwaki self-assigned this Sep 30, 2021
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
Development

Successfully merging a pull request may close this issue.

1 participant