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

Latest Merge Defers Already Deferred Replies #15

Closed
thoughts3rased opened this issue Sep 23, 2021 · 4 comments
Closed

Latest Merge Defers Already Deferred Replies #15

thoughts3rased opened this issue Sep 23, 2021 · 4 comments

Comments

@thoughts3rased
Copy link
Contributor

thoughts3rased commented Sep 23, 2021

The change that was made to the module earlier still errors out when dealing with an interaction with a deferred reply that's passed through, since embed generation will take place before the function is called, making passing an undeferred interaction unfeasable in situations that need it. deferUpdate() might be worth a shot.

@thoughts3rased
Copy link
Contributor Author

Line 26 is the main line of focus here, where deferReply() is called

@ryzyx
Copy link
Owner

ryzyx commented Sep 23, 2021

I havent completely worked on it yet. Its getting late for me imma work on it later

@ryzyx
Copy link
Owner

ryzyx commented Sep 23, 2021

You could also make another pr if you have a stable working code

@ryzyx ryzyx closed this as completed Sep 24, 2021
@ryzyx
Copy link
Owner

ryzyx commented Sep 24, 2021

Latest version has been tested and published to npm, you can use npm i discordjs-button-pagination@interaction or npm i discordjs-button-pagination@latest to get the new version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants