Merge pull request #127 from tauri-apps/feat/disable-role-reaction

feat: disabled role reaction

Author: lorenzolewis

src/events/messageCreate.ts

@@ -1,5 +1,4 @@
 import {
-    MessageOptions,
     ActionRowBuilder,
     ButtonBuilder,
     ThreadChannel,
@@ -15,7 +14,7 @@ import { add_thread_prefix } from '../utils/threads';
 
 export default event({
     name: 'messageCreate',
-    run: async ({}, message) => {
+    run: async ({ }, message) => {
         // Rules for whether or not the message should be dealt with by the bot
         const should_ignore =
             message.author.bot ||
@@ -62,7 +61,7 @@ export default event({
                         `> Reply by <@${message.author.id}>:\n` +
                         message.content,
                     files: message.attachments.map((x) => x),
-                } as MessageOptions;
+                };
                 // Send the message to the thread
                 await thread.send(forwardMessage);
                 // Create response message to the user with a link to the thread
@@ -106,7 +105,7 @@ async function send_instruction_message(thread: ThreadChannel) {
         : base_description;
 
     // Add the solve button to the message
-    const msg = wrap_in_embed(description) as MessageOptions;
+    const msg = wrap_in_embed(description) as any;
     const row = new ActionRowBuilder<ButtonBuilder>().addComponents(
         new ButtonBuilder()
             .setCustomId('solve')

src/events/messageReactionAdd.ts

@@ -1,25 +1,29 @@
 import { ThreadChannel } from 'discord.js';
 import { event } from 'jellycommands';
-import { MESSAGE_READ, REACTION_ROLE_CHANNEL, SOLVABLE_FORUMS } from '../config'
+import {
+    MESSAGE_READ,
+    REACTION_ROLE_CHANNEL,
+    SOLVABLE_FORUMS,
+} from '../config';
 import { hasPermission } from '../utils/reactionHandler';
 
 export default event({
     name: 'messageReactionAdd',
     run: async (_, reaction, user) => {
         try {
             // The bot shouldn't react to its own reactions
-            if (user.bot || SOLVABLE_FORUMS.includes(reaction.message.channelId))
-                return
-            // If this is a reaction in the role reaction channel
-            if (reaction.message.channelId === REACTION_ROLE_CHANNEL) {
-                const result = await hasPermission(reaction, user);
-                result.member.roles.add(result.roleId);
-            } else {
-                // Otherwise we make a more general guess that if the original message was from the bot
-                // and it's a MESSAGE_READ emoji reaction, we assume the message should be deleted
-                if (reaction.message.author.bot && reaction.emoji.name === MESSAGE_READ) {
-                    reaction.message.delete()
-                }
+            if (
+                user.bot ||
+                SOLVABLE_FORUMS.includes(reaction.message.channelId)
+            )
+                return;
+            // Make a general guess that if the original message was from the bot
+            // and it's a MESSAGE_READ emoji reaction, we assume the message should be deleted
+            if (
+                reaction.message.author.bot &&
+                reaction.emoji.name === MESSAGE_READ
+            ) {
+                reaction.message.delete();
             }
         } catch (error) {
             // Couldn't get it to raise the unwanted error in the test server, so instead of

src/events/messageReactionRemove.ts

@@ -8,7 +8,6 @@ import {
     User,
 } from 'discord.js';
 import { REACTION_ROLE, REACTION_ROLE_CHANNEL } from '../config';
-import { wrap_in_embed } from './embed_helpers';
 
 export async function hasPermission(
     reaction: MessageReaction | PartialMessageReaction,
@@ -76,14 +75,12 @@ export async function sendReactionRoleMessage(client: Client) {
         );
         messageArray.push('\n**<#879007560429088800>**');
         messageArray.push(
-            "If you'd like to get involved with Tauri development you can react to the Tauri Contributor role below to be able to chat in the various channels.",
+            "Get involved with Tauri development and browse the different projects.",
         );
         messageArray.push('\n**<#683637724116418561>**');
         messageArray.push(
             "See what the community is working on outside of Tauri. Reach out if you have a passion project you'd like to talk about.",
         );
-        messageArray.push('\n**Server Roles**');
-        messageArray.push('React below to receive the relative role');
 
         let messageBody = messageArray.join('\n');
 
@@ -93,71 +90,24 @@ export async function sendReactionRoleMessage(client: Client) {
 
         console.debug('Got messages');
 
+        // Get an existing message with identical contents
         let message = messages
             .filter((item) => item.content === messageBody)
             .last();
 
         if (message && message.author.id == message.client.user.id) {
             console.debug('Attempting to edit message...');
+            // Edit the message
             await message.edit(messageBody);
             console.debug('Message edited');
         } else {
+            // Delete old messages from the bot
+            messages.filter((item) => item.author.id == item.client.user.id).forEach((item) => item.delete())
             console.debug('Attempting to send message...');
+            // Send the message
             message = await channel.send(messageBody);
             console.debug('Message sent');
         }
-
-        // Loop all available reaction roles
-        REACTION_ROLE.forEach(async (role) => {
-            // Get the reaction
-            console.debug('Attempting to get reactions...');
-            const reaction = await message.reactions.resolve(role.emojiId);
-            console.debug('Got reactions');
-
-            // No reactions yet
-            if (!reaction) {
-                return;
-            }
-
-            // Get all users that reacted minus the bot
-            console.debug('Getting users who reacted...');
-            const reactedUsers = (await reaction.users.fetch()).filter(
-                (user) => user.id !== message.author.id,
-            );
-            console.debug('Finished fetching users');
-
-            // Loop all users and add the role
-            let counter = 0;
-            reactedUsers.forEach(async (user) => {
-                try {
-                    const result = await hasPermission(reaction, user);
-                    counter += 1;
-                    const localCounter = counter;
-                    console.debug(
-                        `(${localCounter} / ${reactedUsers.size}) Attempting to add role...`,
-                    );
-                    if (result) {
-                        await result.member.roles.add(result.roleId);
-                        console.debug(
-                            `(${localCounter} / ${reactedUsers.size}) Role added`,
-                        );
-                    }
-                } catch (error) {
-                    console.error(`Issue adding role: ${error}`);
-                }
-            });
-        });
-
-        var roleDescription: string[] = [];
-
-        REACTION_ROLE.forEach(async (reaction) => {
-            message.react(reaction.emojiId);
-            roleDescription.push(
-                `<:${reaction.emojiName}:${reaction.emojiId}> ${reaction.description}`,
-            );
-        });
-
-        await message.edit(wrap_in_embed(roleDescription.join('\n')));
     } catch (error) {
         console.error(`Issue starting up reaction: ${error}`);
     }