From 0f4b23b4ab641833882f2341ee612b45b6948a8b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Gro=C3=9F?= Date: Tue, 14 Mar 2023 16:06:25 +0100 Subject: [PATCH] Fix registration check causing duplicate acks Registration reminders are now sent as a unique message. We also no longer perform a registration check on autocomplete requests --- events/autocompleteRequest.js | 2 +- util/users.js | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/events/autocompleteRequest.js b/events/autocompleteRequest.js index a9bc38d..238e547 100644 --- a/events/autocompleteRequest.js +++ b/events/autocompleteRequest.js @@ -5,9 +5,9 @@ const { Character, Group } = require('../models'); module.exports = { name: "interactionCreate", async execute (interaction) { + if (interaction.type !== InteractionType.ApplicationCommandAutocomplete) return; let isRegistered = await UserUtils.registrationCheck(interaction); if (!isRegistered) return; - if (interaction.type !== InteractionType.ApplicationCommandAutocomplete) return; console.log(`Autocomplete request from ${interaction.user.tag} (${interaction.user.id}) for ${interaction.commandName} with ${interaction.options.getFocused(true).value}`); let user = await UserUtils.getUserByDiscordId(interaction.member.id); diff --git a/util/users.js b/util/users.js index 9dd006c..a783079 100644 --- a/util/users.js +++ b/util/users.js @@ -22,9 +22,8 @@ module.exports = { return true; } - interaction.reply({ - content: `${interaction.member} You are not registered, use the /register command`, - ephemeral: false + interaction.channel.send({ + content: `${interaction.member} You are not registered, use the /register command` }); return false;