External/card-drop
5
0
Fork 0
Code Issues 45 Pull requests Projects 1 Releases 28 Packages Wiki Activity Actions

Create list effects command #412

Merged
Vylpes merged 15 commits from feature/379-list-effects into develop 2024-12-07 22:32:20 +00:00
Conversation 0 Commits 15 Files changed 10 +5 -4
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 2a7006229a - Show all commits

9
src/helpers/EffectHelper.ts
View file

@ -60,7 +60,7 @@ export default class EffectHelper {
const effects = query[0];
const count = query[1];
const isLastPage = Math.ceil(count / itemsPerPage) - 1 == page;
const totalPages = count > 0 ? Math.ceil(count / itemsPerPage) : 1;
let description = "*none*";
@ -71,7 +71,8 @@ export default class EffectHelper {
const embed = new EmbedBuilder()
.setTitle("Effects")
.setDescription(description)
.setColor(EmbedColours.Ok);
.setColor(EmbedColours.Ok)
.setFooter({ text: `Page ${page} of ${totalPages}` });
const row = new ActionRowBuilder<ButtonBuilder>()
.addComponents(
@ -79,12 +80,12 @@ export default class EffectHelper {
.setCustomId(`effects list ${page - 1}`)
.setLabel("Previous")
.setStyle(ButtonStyle.Primary)
.setDisabled(page == 0),
.setDisabled(page - 1 == 0),
Vylpes marked this conversation as resolved Outdated
Copilot commented 2024-12-06 18:12:26 +00:00
Outdated
Review
Copy link

The 'Previous' button should be disabled when 'page' is 1, not 0.

The 'Previous' button should be disabled when 'page' is 1, not 0.
new ButtonBuilder()
.setCustomId(`effects list ${page + 1}`)
.setLabel("Next")
.setStyle(ButtonStyle.Primary)
.setDisabled(isLastPage),
.setDisabled(page == totalPages),
);
return {