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

Add REST-based interaction handling #8

Open
d4n3436 opened this issue Nov 19, 2021 · 1 comment
Open

Add REST-based interaction handling #8

d4n3436 opened this issue Nov 19, 2021 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@d4n3436
Copy link
Owner

d4n3436 commented Nov 19, 2021

Waiting for a new Discord.Net Labs release to work on it.

Related: Discord-Net-Labs/Discord.Net-Labs@32f5661

@d4n3436 d4n3436 added the enhancement New feature or request label Nov 19, 2021
@d4n3436
Copy link
Owner Author

d4n3436 commented Dec 24, 2021

Any update on this?

@TheStachelfisch you deleted your comment but I will answer it anyway:

REST/HTTP interactions are complex, supporting it implies changing the methods and internal callbacks to support IDiscordInteraction and IComponentInteraction, plus a new delegate for responding REST interactions has to be added.

I've finished adapting the library to support REST interactions, but I have not tested it yet.
I need some feedback/opinions on the way of handling the REST interactions. Currently, the lib uses a callback from the config (RestResponseCallback), which is called when responding/deferring incoming interactions from InteractiveService.HandleRestInteractionAsync, but I'm not sure if this design is the best for this case.

REST interactions branch: https://github.com/d4n3436/Fergun.Interactive/tree/rest-interactions

Feel free to leave any feedback here.

@d4n3436 d4n3436 added the help wanted Extra attention is needed label Dec 29, 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 help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant