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

Method showPageIndicator() Has No Effect #4

Closed
dhkatz opened this issue Jun 10, 2018 · 7 comments
Closed

Method showPageIndicator() Has No Effect #4

dhkatz opened this issue Jun 10, 2018 · 7 comments
Labels
bug/error Something isn't working/right fixed This is not a problem anymore

Comments

@dhkatz
Copy link

dhkatz commented Jun 10, 2018

new paginator.Embeds()
    .setArray(embeds)
    .setAuthorizedUser(message.author)
    .setChannel(message.channel)
    .showPageIndicator(false)
    .build();

Even it I put false explicitly or leave it out the page indication still seems to show.

This seems to be because in Embeds you check for this.showPageIndicator.

const shouldIndicate = this.showPageIndicator

But in PaginationEmbed, showPageIndicator() is actually setting this.pageIndicator

this.pageIndicator = boolean;

@KyleBoyer
Copy link
Contributor

Solved by PR #7

@dhkatz
Copy link
Author

dhkatz commented Jun 12, 2018

Many thanks

@dhkatz
Copy link
Author

dhkatz commented Jun 12, 2018

Also, I noticed when there is only one page the page number shows as null

Edit: Didn't actually read the PR, noticed you changed that too

@gazmull gazmull added the bug/error Something isn't working/right label Jun 13, 2018
@gazmull
Copy link
Owner

gazmull commented Jun 13, 2018

This bug has been fixed by the latest push on v11 branch (I assume you're using v11 branch based on the source code you have written above). Thanks to both of you!

@gazmull gazmull added the fixed This is not a problem anymore label Jun 13, 2018
@gazmull gazmull closed this as completed Jun 13, 2018
@ogaseb
Copy link

ogaseb commented Aug 20, 2019

Sorry for opening again this old issue, but the situation is actually the same as before, when there is only one page it shows null and .setPageIndicator() doesnt seem to work. Im on 1.0.1 version of your plugin and 11.5.1 of discord.js.

@gazmull
Copy link
Owner

gazmull commented Aug 21, 2019

@ProPanek Hello. Thanks for the heads up. I'll re-open this issue until it's verified and fixed.

@gazmull gazmull reopened this Aug 21, 2019
@gazmull gazmull removed the fixed This is not a problem anymore label Aug 21, 2019
gazmull added a commit that referenced this issue Aug 21, 2019
@gazmull
Copy link
Owner

gazmull commented Aug 21, 2019

Fix pushed to NPM. (version 1.0.2)

@gazmull gazmull closed this as completed Aug 21, 2019
@gazmull gazmull added the fixed This is not a problem anymore label Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/error Something isn't working/right fixed This is not a problem anymore
Projects
None yet
Development

No branches or pull requests

4 participants