Don't update public sheet with pilot if checkout failed

sgoodrow · Jun 17, 2024 · 900a90f · 900a90f
1 parent 601363c
commit 900a90f
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 24 deletions.
diff --git a/src/.DS_Store b/src/.DS_Store
diff --git a/src/features/raid-bots/request-subcommand.ts b/src/features/raid-bots/request-subcommand.ts
@@ -50,6 +50,30 @@ Password: ${spoiler(details.password)}
         response += `** Please note that ${currentPilot} is marked as the pilot of ${name} and you may not be able to log in. Your name will not be added as the botpilot in the public bot sheet! **\n\n`;
       }
       response += `The credentials for ${name} have been DM'd to you. Please remember to use \`/bot park\` when you are done!`;
+
+      const logMsg = await thread.send("OK");
+      logMsg.edit(`${status} ${interaction.user} access to ${name}.`);
+
+      if (await publicAccounts.isBotPublic(name)) {
+        try {
+          const guildUser = await interaction.guild?.members.fetch(
+            interaction.user.id
+          );
+
+          if (!currentPilot) {
+            await publicAccounts.updateBotRowDetails(name, {
+              [BOT_SPREADSHEET_COLUMNS.CurrentPilot]:
+                guildUser?.user.username || "UNKNOWN USER",
+              [BOT_SPREADSHEET_COLUMNS.CheckoutTime]: moment().toString(),
+            });
+          }
+        } catch (err) {
+          throw new Error(
+            "Failed to update public record, check the configuration"
+          );
+        }
+      }
+
       await interaction.editReply(response);
     } catch (err) {
       status = "❌ Denied";
@@ -58,30 +82,6 @@ Password: ${spoiler(details.password)}
         `You do not have the correct permissions to access ${name}.`
       );
     }
-
-    const logMsg = await thread.send("OK");
-    logMsg.edit(`${status} ${interaction.user} access to ${name}.`);
-
-    // Update public record
-    if (await publicAccounts.isBotPublic(name)) {
-      try {
-        const guildUser = await interaction.guild?.members.fetch(
-          interaction.user.id
-        );
-
-        if (!currentPilot) {
-          await publicAccounts.updateBotRowDetails(name, {
-            [BOT_SPREADSHEET_COLUMNS.CurrentPilot]:
-              guildUser?.user.username || "UNKNOWN USER",
-            [BOT_SPREADSHEET_COLUMNS.CheckoutTime]: moment().toString(),
-          });
-        }
-      } catch (err) {
-        throw new Error(
-          "Failed to update public record, check the configuration"
-        );
-      }
-    }
   }
 
   public get command() {