Make host a required param when getting settings

givepraise · Dec 15, 2023 · 314d786 · 314d786
1 parent 9defd2a
commit 314d786
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/packages/discord-bot/src/handlers/praise.ts b/packages/discord-bot/src/handlers/praise.ts
@@ -164,7 +164,8 @@ export const praiseHandler: CommandHandler = async (
       }
     } else {
       const directQuantificationEnanbled = (await getSetting(
-        'DISCORD_BOT_DIRECT_PRAISE_QUANTIFICATION_ENABLED'
+        'DISCORD_BOT_DIRECT_PRAISE_QUANTIFICATION_ENABLED',
+        host
       )) as boolean;
 
       // If direct quantification is disabled, give praise directly
@@ -186,7 +187,8 @@ export const praiseHandler: CommandHandler = async (
 
       // If direct quantification is enabled, allow user to select a score from a dropdown
       const allowedScores = (await getSetting(
-        'PRAISE_QUANTIFY_ALLOWED_VALUES'
+        'PRAISE_QUANTIFY_ALLOWED_VALUES',
+        host
       )) as number[];
 
       const select = new StringSelectMenuBuilder()

diff --git a/packages/discord-bot/src/utils/renderMessage.ts b/packages/discord-bot/src/utils/renderMessage.ts
@@ -18,6 +18,8 @@ export const renderMessage = async (
   if (key === 'DM_ERROR' && !host)
     return "The bot can't be used in DMs, please use commands in the server.";
 
+  if (!host) return "Unknown host, can't render message";
+
   let msg = (await getSetting(key, host)) as string;
 
   if (subs) {

diff --git a/packages/discord-bot/src/utils/settingsUtil.ts b/packages/discord-bot/src/utils/settingsUtil.ts
@@ -59,13 +59,12 @@ export const getDefaultSetting = (
 
 export const getSetting = async (
   key: string,
-  host?: string
+  host: string
 ): Promise<string | string[] | boolean | number | number[] | undefined> => {
   const setting = await apiGet<Setting[]>(`/settings?key=${key}`, {
     headers: host ? { host: host } : {},
   })
     .then((res) => {
-      logger.debug(res.data);
       return res.data[0].valueRealized
         ? res.data[0].valueRealized
         : getDefaultSetting(key);