From 5e87f184b6447a8888f1788b3eb59cac813e04b1 Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:29:38 -0700
Subject: [PATCH 1/6] allow max rows for textarea

---
 packages/ui/src/components/textarea.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/ui/src/components/textarea.tsx b/packages/ui/src/components/textarea.tsx
index 0cf135e88..16d6207b2 100644
--- a/packages/ui/src/components/textarea.tsx
+++ b/packages/ui/src/components/textarea.tsx
@@ -13,6 +13,7 @@ type TextareaProps = Pick<
   | 'defaultValue'
   | 'id'
   | 'maxLength'
+  | 'maxRows'
   | 'minLength'
   | 'minRows'
   | 'name'

From 7abe2fecd118b37f282a39857c40f873f13fa4bc Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:30:31 -0700
Subject: [PATCH 2/6] remove students

---
 .../app/routes/_dashboard.students.remove.tsx | 140 ++++++++++++++++++
 1 file changed, 140 insertions(+)
 create mode 100644 apps/admin-dashboard/app/routes/_dashboard.students.remove.tsx

diff --git a/apps/admin-dashboard/app/routes/_dashboard.students.remove.tsx b/apps/admin-dashboard/app/routes/_dashboard.students.remove.tsx
new file mode 100644
index 000000000..e89eede82
--- /dev/null
+++ b/apps/admin-dashboard/app/routes/_dashboard.students.remove.tsx
@@ -0,0 +1,140 @@
+import {
+  type ActionFunctionArgs,
+  data,
+  Form,
+  type LoaderFunctionArgs,
+  redirect,
+  useActionData,
+} from 'react-router';
+import z from 'zod';
+
+import { job } from '@oyster/core/bull';
+import { db } from '@oyster/db';
+import {
+  Button,
+  ErrorMessage,
+  Field,
+  getErrors,
+  Modal,
+  Textarea,
+  validateForm,
+} from '@oyster/ui';
+import { Callout } from '@oyster/ui/callout';
+
+import { Route } from '@/shared/constants';
+import {
+  commitSession,
+  ensureUserAuthenticated,
+  toast,
+} from '@/shared/session.server';
+
+export async function loader({ request }: LoaderFunctionArgs) {
+  await ensureUserAuthenticated(request);
+
+  return {};
+}
+
+const RemoveMembersFormData = z.object({
+  memberIds: z
+    .string()
+    .min(1)
+    .transform((value) => value.split('\n').filter(Boolean)),
+});
+
+export async function action({ request }: ActionFunctionArgs) {
+  const session = await ensureUserAuthenticated(request);
+
+  const form = await request.formData();
+
+  const result = await validateForm(form, RemoveMembersFormData);
+
+  if (!result.ok) {
+    return data(result, { status: 400 });
+  }
+
+  const count = await removeMembers(result.data.memberIds);
+
+  toast(session, {
+    message: `Removed ${count} members.`,
+  });
+
+  return redirect(Route['/students'], {
+    headers: {
+      'Set-Cookie': await commitSession(session),
+    },
+  });
+}
+
+async function removeMembers(ids: string[]): Promise<number> {
+  const students = await db
+    .deleteFrom('students')
+    .where('id', 'in', ids)
+    .returning(['airtableId', 'email', 'firstName', 'slackId'])
+    .execute();
+
+  for (const student of students) {
+    job('student.removed', {
+      airtableId: student.airtableId as string,
+      email: student.email,
+      firstName: student.firstName,
+      sendViolationEmail: false,
+      slackId: student.slackId,
+    });
+  }
+
+  return students.length;
+}
+
+export default function RemoveMembersPage() {
+  const { error, errors } = getErrors(useActionData<typeof action>());
+
+  return (
+    <Modal onCloseTo={Route['/students']}>
+      <Modal.Header>
+        <Modal.Title>Bulk Remove Members</Modal.Title>
+        <Modal.CloseButton />
+      </Modal.Header>
+
+      <Modal.Description>
+        This action is not reversible. All of their engagement records will be
+        deleted and they will be removed from Slack, Mailchimp and Airtable.
+      </Modal.Description>
+
+      <Callout color="blue">
+        Note: These members will immediately be removed from our database, but
+        it may take some time for them to be removed from Slack, Mailchimp and
+        Airtable.
+      </Callout>
+
+      <Form className="form" method="post">
+        <ErrorMessage>{error}</ErrorMessage>
+
+        <Field
+          description="Please list the IDs of the members to remove separated by a newline."
+          error={errors.memberIds}
+          label="Member IDs"
+          labelFor="memberIds"
+          required
+        >
+          <Textarea
+            id="memberIds"
+            maxRows={10}
+            minRows={10}
+            name="memberIds"
+            required
+          />
+        </Field>
+
+        <Button.Group>
+          <Button color="error" type="submit">
+            Remove
+          </Button>
+        </Button.Group>
+      </Form>
+    </Modal>
+  );
+}
+
+export function ErrorBoundary() {
+  return <></>;
+}

From b116935e4596b83df5225e2f545e0bfac84cdd0a Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:35:03 -0700
Subject: [PATCH 3/6] entrypoint in the students page

---
 .../app/routes/_dashboard.students.tsx        | 40 ++++++++++++++++++-
 apps/admin-dashboard/app/shared/constants.ts  |  1 +
 2 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/apps/admin-dashboard/app/routes/_dashboard.students.tsx b/apps/admin-dashboard/app/routes/_dashboard.students.tsx
index bf26212f3..35b104d46 100644
--- a/apps/admin-dashboard/app/routes/_dashboard.students.tsx
+++ b/apps/admin-dashboard/app/routes/_dashboard.students.tsx
@@ -1,6 +1,14 @@
 import dayjs from 'dayjs';
 import { sql } from 'kysely';
-import { CornerUpLeft, Edit, ExternalLink, Star, Trash } from 'react-feather';
+import {
+  CornerUpLeft,
+  Edit,
+  ExternalLink,
+  Menu,
+  Star,
+  Trash,
+  Trash2,
+} from 'react-feather';
 import {
   generatePath,
   Link,
@@ -14,6 +22,7 @@ import { db } from '@oyster/db';
 import {
   Dashboard,
   Dropdown,
+  IconButton,
   Pagination,
   type SerializeFrom,
   Table,
@@ -118,6 +127,10 @@ export default function StudentsPage() {
 
       <Dashboard.Subheader>
         <Dashboard.SearchForm placeholder="Search by name..." />
+
+        <div className="ml-auto flex items-center gap-2">
+          <StudentsMenuDropdown />
+        </div>
       </Dashboard.Subheader>
 
       <StudentsTable />
@@ -127,6 +140,31 @@ export default function StudentsPage() {
   );
 }
 
+function StudentsMenuDropdown() {
+  return (
+    <Dropdown.Root>
+      <Dropdown.Trigger>
+        <IconButton
+          backgroundColor="gray-100"
+          backgroundColorOnHover="gray-200"
+          icon={<Menu />}
+          shape="square"
+        />
+      </Dropdown.Trigger>
+
+      <Dropdown>
+        <Dropdown.List>
+          <Dropdown.Item>
+            <Link to={Route['/students/remove']}>
+              <Trash2 /> Remove Members
+            </Link>
+          </Dropdown.Item>
+        </Dropdown.List>
+      </Dropdown>
+    </Dropdown.Root>
+  );
+}
+
 type StudentInView = SerializeFrom<typeof loader>['students'][number];
 
 function StudentsTable() {
diff --git a/apps/admin-dashboard/app/shared/constants.ts b/apps/admin-dashboard/app/shared/constants.ts
index 0f88f99dc..3b6701821 100644
--- a/apps/admin-dashboard/app/shared/constants.ts
+++ b/apps/admin-dashboard/app/shared/constants.ts
@@ -41,6 +41,7 @@ const ROUTES = [
   '/schools/:id/chapter/create',
   '/schools/:id/edit',
   '/students',
+  '/students/remove',
   '/students/:id/email',
   '/students/:id/points/grant',
   '/students/:id/remove',

From abd2d8b71134729375b4e6f0da7dd5d0bfb1c4ef Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:40:32 -0700
Subject: [PATCH 4/6] update default bull options

---
 packages/core/src/infrastructure/bull.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/core/src/infrastructure/bull.ts b/packages/core/src/infrastructure/bull.ts
index 37d6334d9..642fc5959 100644
--- a/packages/core/src/infrastructure/bull.ts
+++ b/packages/core/src/infrastructure/bull.ts
@@ -44,10 +44,10 @@ export function getQueue(name: string) {
     _queues[name] = new Queue(name, {
       connection,
       defaultJobOptions: {
-        attempts: 3,
+        attempts: 1,
         backoff: { delay: 5000, type: 'exponential' },
-        removeOnComplete: { age: 60 * 60 * 24 * 1, count: 100 },
-        removeOnFail: { age: 60 * 60 * 24 * 7, count: 1000 },
+        removeOnComplete: { age: 60 * 60 * 24 * 1, count: 1000 },
+        removeOnFail: { age: 60 * 60 * 24 * 30, count: 10000 },
       },
     });
   }
@@ -143,7 +143,7 @@ export function registerWorker<Schema extends ZodType>(
     autorun: false,
     connection: redis,
     removeOnComplete: { age: 60 * 60 * 24 * 1, count: 1000 },
-    removeOnFail: { age: 60 * 60 * 24 * 7, count: 10000 },
+    removeOnFail: { age: 60 * 60 * 24 * 30, count: 10000 },
     ...options,
   };
 

From d5b53481bac4c39f8ad25d201feab417ac44e375 Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:44:00 -0700
Subject: [PATCH 5/6] update language

---
 apps/admin-dashboard/app/routes/_dashboard.students.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/apps/admin-dashboard/app/routes/_dashboard.students.tsx b/apps/admin-dashboard/app/routes/_dashboard.students.tsx
index 35b104d46..63f588e19 100644
--- a/apps/admin-dashboard/app/routes/_dashboard.students.tsx
+++ b/apps/admin-dashboard/app/routes/_dashboard.students.tsx
@@ -156,7 +156,7 @@ function StudentsMenuDropdown() {
         <Dropdown.List>
           <Dropdown.Item>
             <Link to={Route['/students/remove']}>
-              <Trash2 /> Remove Members
+              <Trash2 /> Bulk Remove Members
             </Link>
           </Dropdown.Item>
         </Dropdown.List>

From 4f8f10a027f65a276419f4386ecee21a6c50298c Mon Sep 17 00:00:00 2001
From: Rami Abdou <ramiabdou98@gmail.com>
Date: Tue, 23 Sep 2025 09:50:07 -0700
Subject: [PATCH 6/6] ensure that airtable delete throws

---
 packages/core/src/modules/airtable.ts | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/packages/core/src/modules/airtable.ts b/packages/core/src/modules/airtable.ts
index f3e404f94..07e6210b6 100644
--- a/packages/core/src/modules/airtable.ts
+++ b/packages/core/src/modules/airtable.ts
@@ -14,6 +14,7 @@ import {
   AirtableBullJob,
   type GetBullJobData,
 } from '@/infrastructure/bull.types';
+import { reportException } from '@/infrastructure/sentry';
 import { IS_PRODUCTION } from '@/shared/env';
 import { ColorStackError, ErrorWithContext } from '@/shared/errors';
 import { RateLimiter } from '@/shared/utils/rate-limiter';
@@ -387,7 +388,7 @@ async function deleteAirtableRecord({
 
   await airtableRateLimiter.process();
 
-  await fetch(
+  const response = await fetch(
     `${AIRTABLE_API_URI}/${airtableBaseId}/${airtableTableId}/${airtableRecordId}`,
     {
       headers: getAirtableHeaders(),
@@ -395,6 +396,20 @@ async function deleteAirtableRecord({
     }
   );
 
+  if (!response.ok) {
+    const error = new ErrorWithContext(
+      'Failed to delete Airtable record.'
+    ).withContext({
+      airtableBaseId,
+      airtableRecordId,
+      airtableTableId,
+    });
+
+    reportException(error);
+
+    throw error;
+  }
+
   console.log({
     code: 'airtable_record_deleted',
     message: 'Airtable record was deleted.',