Fix past job post loading on startup (#436)

* Fix isStaff check * Filter staff messages out of job board startup * Fix message loading * Track post time when deleted, as intended
reactiflux · Jan 17, 2025 · 73a4aa2 · 73a4aa2
1 parent ca15ef6
commit 73a4aa2
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/src/features/jobs-moderation/job-mod-helpers.ts b/src/features/jobs-moderation/job-mod-helpers.ts
@@ -97,9 +97,9 @@ export const loadJobs = async (bot: Client, channel: TextChannel) => {
     !oldestMessage ||
     differenceInDays(now, oldestMessage.createdAt) < DAYS_OF_POSTS
   ) {
-    const messages = await channel.messages.fetch({
-      ...(oldestMessage ? { after: oldestMessage.message.id } : {}),
-    });
+    const messages = await channel.messages.fetch(
+      oldestMessage ? { before: oldestMessage.message.id } : {},
+    );
     console.log(
       "[DEBUG] loadJobs()",
       `Oldest message: ${oldestMessage ? oldestMessage.createdAt : "none"}.`,
@@ -135,18 +135,19 @@ export const loadJobs = async (bot: Client, channel: TextChannel) => {
     }
     oldestMessage = newMessages
       .sort((a, b) => compareAsc(a.createdAt, b.createdAt))
-      .at(-1);
+      .at(0);
     if (!oldestMessage) break;
 
-    const humanMessages = newMessages.filter(
+    const humanNonStaffMessages = newMessages.filter(
       (m) =>
         differenceInDays(now, m.createdAt) < DAYS_OF_POSTS &&
         !m.message.system &&
+        !isStaff(m.message.member) &&
         m.authorId !== bot.user?.id,
     );
     const [hiring, forHire] = partition(
       (m) => m.type === PostType.hiring,
-      humanMessages,
+      humanNonStaffMessages,
     );
 
     jobBoardMessageCache = { hiring, forHire };

diff --git a/src/helpers/discord.ts b/src/helpers/discord.ts
@@ -22,7 +22,7 @@ import {
 export const difference = <T>(a: Set<T>, b: Set<T>) =>
   new Set(Array.from(a).filter((x) => !b.has(x)));
 
-const staffRoles = ["mvp", "moderator", "admin", "admins"];
+const staffRoles = ["moderator", "admin", "admins"];
 const helpfulRoles = ["mvp", "star helper"];
 
 const hasRole = (member: GuildMember, roles: string | string[]) =>

diff --git a/src/helpers/modLog.ts b/src/helpers/modLog.ts
@@ -163,7 +163,7 @@ ${reportedMessage}
 
 ${reportedMessage}`;
     case ReportReasons.jobAge:
-      return `${preface}, for hire post expired.`;
+      return `${preface}, for hire post expired. ${extra}`;
     case ReportReasons.jobFrequency:
       return `${preface}, posting too frequently.`;