ygimenez/Pagination-Utils

Pagination not working in Slash Commands

aarondyejr opened this issue · 15 comments

Checklist

Make sure that you've checked all the items below.

  • Bot has the following permissions:
    • MESSAGE_ADD_REACTION
    • MESSAGE_EXT_EMOJI
    • MESSAGE_READ/WRITE
    • VIEW_CHANNEL
  • If using JDABuilder.createLight(), you added the following gateway intents:
    • GUILD_MESSAGES
    • GUILD_MESSAGE_REACTIONS
  • If using .setRemoveOnReact(true), you have the following permission:
    • MESSAGE_MANAGE
  • PaginationUtils is up-to-date.
  • You have activated the library as descripted in the README.

Library info

What libraries versions are you using.

  • JDA version 757de9b
  • PaginationUtils version 2.2.0

Describe the bug

A clear and concise description of what the bug is.

When trying to use the pagination on an embed through slash commands I get 429 errors saying the reaction won't work.
I have tried countless times thinking it was my fault and it very well still could be but I have no idea what to change if it is so I am starting to think it is a bug.

To Reproduce

Steps to reproduce the behavior:

  1. Follow the readme
  2. Send the embed via:
event.hook.sendMessageEmbeds(...).queue { it ->
    Pages.pagination(it, myPages)
}
  1. Call the slash command
  2. See error

Expected behavior

A clear and concise description of what you expected to happen.

Well I expected it to react to the embed and work as normal.

Screenshots

If applicable, add screenshots to help explain your problem.

image

Additional context

Add any other context about the problem here.

Do the reactions appear? That seems like you're getting ratelimited somewhere. Could you try on a private bot please?

@ygimenez They do not appear at all and no matter on what bot I try it on the same thing happens.

Slash commands don't need gateway intents. In fact in the example reported on github the JDA configuration is the following (SlashBotExample):

JDABuilder.createLight(getBottoken(), EnumSet.noneOf(GatewayIntent.class))

If possible try to respect this configuration. (I am also interested in this development)

I'm rewriting the library to be compliant with the new features, including buttons and slash commands.

how's the work going ? do you have an alleged release date?

I've halted development for a while due to personal issues, but I'll return to it as soon as possible.

Has there been anymore work done on this?

For sure, you can check the latest commits to see what else was added, I'm just finishing polishing some rough edges before I release a new version.

Will the pagination now work through buttons in slash commands since you're not really able to get the message easily?

Both types of buttons will be supported, as for slashes all my tests paginate successfully as long as you passed a valid Message object (e.g: the reply message).

Ah okay, I was trying earlier to get a message object through slash commands and was unsuccessful. I hope you'll give some examples 😅 Aside from that, is there an ETA on this next release?

I don't know, maybe by next month or so

Looking forward to it!

Please check whether this issue still exists in latest version.

Considered fixed. Should it be required feel free to reopen the issue.