RabbitLabs/vylbot-app
RabbitLabs/vylbot-app
5
0
Fork 0
Code Issues 90 Pull requests 4 Projects 1 Releases 22 Packages Wiki Activity Actions

#214 Add ephemeral reply upon rules success #231

Closed
Vylpes wants to merge 1 commit from defect/214-application-did-not-respond into develop
pull from: defect/214-application-did-not-respond
merge into: RabbitLabs:develop
Conversation 0 Commits 1 Files changed 1 +10 -3
1 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit c4b5d9671b - Show all commits

13
src/commands/rules.ts
View file

@ -32,13 +32,13 @@ export default class Rules extends Command {
const rules = JSON.parse(rulesFile) as IRules[];
const embeds: EmbedBuilder[] = [];
rules.forEach(rule => {
Vylpes marked this conversation as resolved
VylpesTester commented 2022-12-16 21:17:06 +00:00
Review
Copy link

Should we do anything if rules is empty? What about if the rules fail to be produced?

Should we do anything if rules is empty? What about if the rules fail to be produced?
const embed = new EmbedBuilder()
.setColor(EmbedColours.Ok)
.setTitle(rule.title || "Rules")
.setDescription(rule.description ? rule.description.join("\n") : "*none*");
if (rule.image) {
embed.setImage(rule.image);
}
@ -55,7 +55,14 @@ export default class Rules extends Command {
if (!channel) {
Vylpes marked this conversation as resolved
VylpesTester commented 2022-12-16 21:16:12 +00:00
Review
Copy link

Might be worth adding a failed embed here, just in case and so it doesn't fail

Might be worth adding a failed embed here, just in case and so it doesn't fail
return;
}
await channel.send({ embeds: embeds });
const successEmbed = new EmbedBuilder()
.setColor(EmbedColours.Ok)
.setTitle("Success")
.setDescription("The rules have sent to this channel successfully");
await interaction.reply({ embeds: [ successEmbed], ephemeral: true });
}
}