From 86adad93587617202659a79fa741a52349137aa0 Mon Sep 17 00:00:00 2001
From: Rotimi Best <best@classroomio.com>
Date: Sat, 1 Nov 2025 09:54:19 +0100
Subject: [PATCH 1/3] fix: onboarding validation error showing translation key
 #586 (#593)

* fix: onboarding validation error showing translation key #586

* fix: remove debug log for profile update errors
---
 .../Org/Settings/LanguagePicker.svelte        |   4 +-
 .../components/Org/Settings/Profile.svelte    |  19 +-
 apps/dashboard/src/lib/utils/constants/app.ts |  42 +--
 .../src/lib/utils/functions/validator.ts      | 244 +++++++++---------
 4 files changed, 150 insertions(+), 159 deletions(-)

diff --git a/apps/dashboard/src/lib/components/Org/Settings/LanguagePicker.svelte b/apps/dashboard/src/lib/components/Org/Settings/LanguagePicker.svelte
index d5b52e98..9a30f46d 100644
--- a/apps/dashboard/src/lib/components/Org/Settings/LanguagePicker.svelte
+++ b/apps/dashboard/src/lib/components/Org/Settings/LanguagePicker.svelte
@@ -1,11 +1,11 @@
-<script>
+<script lang="ts">
   import { Dropdown } from 'carbon-components-svelte';
   import { t, initialized } from '$lib/utils/functions/translations';
   import { LANGUAGES } from '$lib/utils/constants/translation';
   import { LOCALE } from '$lib/utils/types';
 
   export let className = '';
-  export let value = LOCALE.EN;
+  export let value: LOCALE = LOCALE.EN;
   export let hasLangChanged = false;
 
   const dropdownItems = [{ id: '', text: 'Pick a Language' }, ...LANGUAGES];
diff --git a/apps/dashboard/src/lib/components/Org/Settings/Profile.svelte b/apps/dashboard/src/lib/components/Org/Settings/Profile.svelte
index dc4f4544..fa4249d1 100644
--- a/apps/dashboard/src/lib/components/Org/Settings/Profile.svelte
+++ b/apps/dashboard/src/lib/components/Org/Settings/Profile.svelte
@@ -1,4 +1,4 @@
-<script>
+<script lang="ts">
   import TextField from '$lib/components/Form/TextField.svelte';
   import { VARIANTS } from '$lib/components/PrimaryButton/constants';
   import PrimaryButton from '$lib/components/PrimaryButton/index.svelte';
@@ -13,14 +13,15 @@
   import { Column, Grid, Row } from 'carbon-components-svelte';
   import SectionTitle from '../SectionTitle.svelte';
   import LanguagePicker from './LanguagePicker.svelte';
+  import { LOCALE } from '$lib/utils/types';
 
   let avatar = '';
   let loading = false;
   let hasLangChanged = false;
-  let locale = '';
+  let locale: LOCALE;
   let hasUnsavedChanges = false;
 
-  let errors = {};
+  let errors: Record<string, string> = {};
 
   async function handleUpdate() {
     errors = updateProfileValidation($profile);
@@ -33,7 +34,7 @@
       console.log({ hasLangChanged });
       loading = true;
 
-      const updates = {
+      const updates: Partial<typeof $profile> = {
         fullname: $profile.fullname,
         username: $profile.username,
         email: $profile.email,
@@ -54,7 +55,8 @@
           updates.avatar_url = response.publicUrl;
           $profile.avatar_url = response.publicUrl;
         }
-        avatar = undefined;
+
+        avatar = '';
       }
 
       let { error } = await supabase.from('profile').update(updates).match({ id: $profile.id });
@@ -127,12 +129,7 @@
         errorMessage={$t(errors.email)}
         onChange={() => (hasUnsavedChanges = true)}
       />
-      <LanguagePicker
-        onChange={() => (hasUnsavedChanges = true)}
-        bind:hasLangChanged
-        bind:value={locale}
-        className="w-full lg:w-60 mb-4"
-      />
+      <LanguagePicker bind:hasLangChanged bind:value={locale} className="w-full lg:w-60 mb-4" />
     </Column>
   </Row>
 
diff --git a/apps/dashboard/src/lib/utils/constants/app.ts b/apps/dashboard/src/lib/utils/constants/app.ts
index d6c1cf70..a96c015f 100644
--- a/apps/dashboard/src/lib/utils/constants/app.ts
+++ b/apps/dashboard/src/lib/utils/constants/app.ts
@@ -1,30 +1,34 @@
 export const blockedSubdomain = [
+  'academy',
   'app',
   'apps',
-  'play',
-  'support',
-  'demo',
-  'help',
-  'teachers',
-  'church',
-  'tech',
+  'blog',
   'build',
+  'campaign',
+  'church',
+  'cloud',
   'community',
   'conference',
-  'campaign',
-  'blog',
-  'forum',
-  'mobile',
+  'course',
+  'courses',
+  'demo',
   'dev',
-  'stage',
-  'prod',
+  'forum',
   'group',
   'groups',
-  'academy',
-  'schools',
-  'training',
-  'cloud',
+  'help',
+  'launch',
   'launchweek',
-  'course',
-  'launch'
+  'mobile',
+  'play',
+  'prod',
+  'production',
+  'schools',
+  'stage',
+  'staging',
+  'support',
+  'teacher',
+  'teachers',
+  'tech',
+  'training'
 ];
diff --git a/apps/dashboard/src/lib/utils/functions/validator.ts b/apps/dashboard/src/lib/utils/functions/validator.ts
index 07acb922..895833ed 100644
--- a/apps/dashboard/src/lib/utils/functions/validator.ts
+++ b/apps/dashboard/src/lib/utils/functions/validator.ts
@@ -2,141 +2,40 @@ import z from 'zod';
 import { get } from 'svelte/store';
 import isNumber from 'lodash/isNumber';
 import { validateEmail } from './validateEmail';
-import { t as _t, initialized } from '$lib/utils/functions/translations';
+import { t, initialized } from '$lib/utils/functions/translations';
 
-const t = {
-  get: (key: string) => {
-    const isInitialized = get(initialized);
-    if (!isInitialized) return key;
-
-    return _t.get(key);
-  }
-};
-
-const getOrgNameValidation = () =>
-  z
+function getOrgNameValidation() {
+  return z
     .string()
     .min(5, { message: `${t.get('validations.organization_name.min_char')}` })
     .refine((val) => !/^[-]|[-]$/.test(val), {
       message: `${t.get('validations.organization_name.hyphen_rule')}`
     });
+}
 
-const getSiteNameValidation = () =>
-  z
+function getSiteNameValidation() {
+  return z
     .string()
     .min(5, { message: `${t.get('validations.site_name.min_char')}` })
     .refine((val) => !/^[-]|[-]$/.test(val), {
       message: `${t.get('validations.site_name.hyphen_rule')}`
     });
-
-const getNewsfeedValidation = () =>
-  z.string().min(5, { message: `${t.get('validations.news_feed.min_char')}` });
-
-const lessonSchema = z.object({
-  title: z.string().nonempty({ message: `${t.get('validations.lesson_schema.empty_title')}` }),
-  lesson_at: z.string().optional(),
-  call_url: z.string().nullable().optional(),
-  is_unlocked: z.boolean().optional()
-});
-
-const createQuizValidationSchema = z.object({
-  title: z.string().min(6, {
-    message: `${t.get('validations.course_payment.full_name.min_char')}`
-  })
-});
-
-const askCommunityValidationSchema = z.object({
-  title: z.string().min(6, {
-    message: `${t.get('validations.ask_community.title.min_char')}`
-  }),
-  body: z.string().min(20, {
-    message: `${t.get('validations.ask_community.body.min_char')}`
-  }),
-  courseId: z.string().min(36, {
-    message: `${t.get('validations.ask_community.course_id.select_course')}`
-  })
-});
-const commentInCommunityValidationSchema = z.object({
-  comment: z.string().min(6, {
-    message: `${t.get('validations.comment_in_community.min_char')}`
-  })
-});
-
-const orgLandingpageValidationSchema = z.object({
-  name: z.string().min(6, {
-    message: `${t.get('validations.org_landing_page.name.min_char')}`
-  }),
-  email: z.string().email({
-    message: `${t.get('validations.org_landing_page.email.invalid_email')}`
-  }),
-  phone: z.string().min(6, {
-    message: `${t.get('validations.org_landing_page.phone.min_char')}`
-  }),
-  message: z.string().min(20, {
-    message: `${t.get('validations.org_landing_page.message.min_char')}`
-  })
-});
-
-const forgotValidationSchema = z.object({
-  email: z.string().email({
-    message: `${t.get('validations.forgot.invalid_email')}`
-  })
-});
-
-const authValidationSchema = z.object({
-  email: z.string().email({
-    message: 'validations.auth.email.invalid_email'
-  }),
-  password: z.string().min(6, {
-    message: 'validations.auth.password.min_char'
-  })
-});
-
-const resetValidationSchema = z.object({
-  password: z.string().min(6, {
-    message: `${t.get('validations.reset.password.min_char')}`
-  })
-});
-
-const onboardingValidationSchema = {
-  stepOne: z.object({
-    fullname: z
-      .string()
-      .min(5, { message: `${t.get('validations.onboarding.step_one.full_name.min_char')}` }),
-    orgName: getOrgNameValidation(),
-    siteName: getSiteNameValidation()
-  }),
-  stepTwo: z.object({
-    goal: z
-      .string({
-        required_error: `${t.get('validations.onboarding.step_two.goal.required')}`
-      })
-      .min(1),
-    source: z
-      .string({
-        required_error: `${t.get('validations.onboarding.step_two.source.required')}`
-      })
-      .min(1)
-  })
-};
-
-const saveCertificateSchema = z.object({
-  description: z.string().max(200, 'course.navItem.certificates.description_error'),
-  is_certificate_downloadable: z.boolean(),
-  certificate_theme: z.string()
-});
+}
+function getNewsfeedValidation() {
+  return z.string().min(5, { message: `${t.get('validations.news_feed.min_char')}` });
+}
 
 export const saveCertificateValidation = (fields = {}) => {
-  const { error } = saveCertificateSchema.safeParse(fields);
+  const schema = z.object({
+    description: z.string().max(200, 'course.navItem.certificates.description_error'),
+    is_certificate_downloadable: z.boolean(),
+    certificate_theme: z.string()
+  });
+
+  const { error } = schema.safeParse(fields);
   return processErrors(error);
 };
 
-const updateProfileValidationSchema = z.object({
-  email: z.string().email({ message: 'validations.user_profile.email' }),
-  username: z.string().nonempty({ message: 'validations.user_profile.username' }),
-  fullname: z.string().min(5, { message: 'validations.user_profile.fullname' })
-});
-
 export const getConfirmPasswordError = ({ password, confirmPassword }) => {
   return password > 6 && confirmPassword > 6 && password !== confirmPassword
     ? `${t.get('validations.confirm_password.not_match')}`
@@ -173,12 +72,30 @@ export const processErrors = (error, mapToId?: boolean) => {
 };
 
 export const authValidation = (fields = {}) => {
-  const { error } = authValidationSchema.safeParse(fields);
+  const schema = z.object({
+    email: z.string().email({
+      message: 'validations.auth.email.invalid_email'
+    }),
+    password: z.string().min(6, {
+      message: 'validations.auth.password.min_char'
+    })
+  });
+
+  const { error } = schema.safeParse(fields);
+
   return processErrors(error);
 };
 
 export const lessonValidation = (lesson = {}) => {
-  const { error } = lessonSchema.safeParse(lesson);
+  const schema = z.object({
+    title: z.string().nonempty({ message: `${t.get('validations.lesson_schema.empty_title')}` }),
+    lesson_at: z.string().optional(),
+    call_url: z.string().nullable().optional(),
+    is_unlocked: z.boolean().optional()
+  });
+
+  const { error } = schema.safeParse(lesson);
+
   return processErrors(error);
 };
 
@@ -197,24 +114,70 @@ export const coursePaymentValidation = (fields = {}) => {
 };
 
 export const resetValidation = (fields = {}) => {
-  const { error } = resetValidationSchema.safeParse(fields);
+  const schema = z.object({
+    password: z.string().min(6, {
+      message: `${t.get('validations.reset.password.min_char')}`
+    })
+  });
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };
 
 export const forgotValidation = (fields = {}) => {
-  const { error } = forgotValidationSchema.safeParse(fields);
+  const schema = z.object({
+    email: z.string().email({
+      message: `${t.get('validations.forgot.invalid_email')}`
+    })
+  });
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };
 
 export const orgLandingpageValidation = (fields = {}) => {
-  const { error } = orgLandingpageValidationSchema.safeParse(fields);
+  const schema = z.object({
+    name: z.string().min(6, {
+      message: `${t.get('validations.org_landing_page.name.min_char')}`
+    }),
+    email: z.string().email({
+      message: `${t.get('validations.org_landing_page.email.invalid_email')}`
+    }),
+    phone: z.string().min(6, {
+      message: `${t.get('validations.org_landing_page.phone.min_char')}`
+    }),
+    message: z.string().min(20, {
+      message: `${t.get('validations.org_landing_page.message.min_char')}`
+    })
+  });
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };
 
 export const onboardingValidation = (fields = {}, step) => {
+  const onboardingValidationSchema = {
+    stepOne: z.object({
+      fullname: z
+        .string()
+        .min(5, { message: `${t.get('validations.onboarding.step_one.full_name.min_char')}` }),
+      orgName: getOrgNameValidation(),
+      siteName: getSiteNameValidation()
+    }),
+    stepTwo: z.object({
+      goal: z
+        .string({
+          required_error: `${t.get('validations.onboarding.step_two.goal.required')}`
+        })
+        .min(1),
+      source: z
+        .string({
+          required_error: `${t.get('validations.onboarding.step_two.source.required')}`
+        })
+        .min(1)
+    })
+  };
+
   const schema =
     step === 1 ? onboardingValidationSchema.stepOne : onboardingValidationSchema.stepTwo;
   const { error } = schema.safeParse(fields);
@@ -223,7 +186,11 @@ export const onboardingValidation = (fields = {}, step) => {
 };
 
 export const updateProfileValidation = (fields = {}) => {
-  const schema = updateProfileValidationSchema;
+  const schema = z.object({
+    email: z.string().email({ message: 'validations.user_profile.email' }),
+    username: z.string().nonempty({ message: 'validations.user_profile.username' }),
+    fullname: z.string().min(5, { message: 'validations.user_profile.fullname' })
+  });
   const { error } = schema.safeParse(fields);
 
   return processErrors(error);
@@ -297,19 +264,42 @@ export const updateProfileEmailValidation = (email) => {
 };
 
 export const createQuizValidation = (fields = {}) => {
-  const { error } = createQuizValidationSchema.safeParse(fields);
+  const schema = z.object({
+    title: z.string().min(6, {
+      message: `${t.get('validations.course_payment.full_name.min_char')}`
+    })
+  });
+
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };
 
 export const askCommunityValidation = (fields = {}) => {
-  const { error } = askCommunityValidationSchema.safeParse(fields);
+  const schema = z.object({
+    title: z.string().min(6, {
+      message: `${t.get('validations.ask_community.title.min_char')}`
+    }),
+    body: z.string().min(20, {
+      message: `${t.get('validations.ask_community.body.min_char')}`
+    }),
+    courseId: z.string().min(36, {
+      message: `${t.get('validations.ask_community.course_id.select_course')}`
+    })
+  });
+
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };
 
 export const commentInCommunityValidation = (fields = {}) => {
-  const { error } = commentInCommunityValidationSchema.safeParse(fields);
+  const schema = z.object({
+    comment: z.string().min(6, {
+      message: `${t.get('validations.comment_in_community.min_char')}`
+    })
+  });
+  const { error } = schema.safeParse(fields);
 
   return processErrors(error);
 };

From ba0fc32a872b661d29dcecbbfaea6770cf43f97b Mon Sep 17 00:00:00 2001
From: Rotimi Best <best@classroomio.com>
Date: Sat, 1 Nov 2025 11:25:32 +0100
Subject: [PATCH 2/3] chore: add a clean command

---
 package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package.json b/package.json
index d6d0e900..a9cf7436 100644
--- a/package.json
+++ b/package.json
@@ -6,7 +6,7 @@
     "start": "turbo run start",
     "dev": "turbo prepare --concurrency 13 && turbo run dev",
     "lint": "turbo run lint",
-    "clean": "turbo clean && rm -rf node_modules",
+    "clean": "turbo clean && find . -name 'node_modules' -type d -prune -exec rm -rf '{}' + && pnpm store prune",
     "format": "prettier . --write",
     "release": "standard-version",
     "ci": "cypress run",

From eb40b6dab6b3d8526608c82334b34ed00cd38a70 Mon Sep 17 00:00:00 2001
From: Rotimi Best <best@classroomio.com>
Date: Sat, 1 Nov 2025 11:53:27 +0100
Subject: [PATCH 3/3] fix: remove message from email verification response

---
 apps/dashboard/src/routes/api/email/verify_email/+server.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/apps/dashboard/src/routes/api/email/verify_email/+server.ts b/apps/dashboard/src/routes/api/email/verify_email/+server.ts
index feff8285..876fb251 100644
--- a/apps/dashboard/src/routes/api/email/verify_email/+server.ts
+++ b/apps/dashboard/src/routes/api/email/verify_email/+server.ts
@@ -39,7 +39,6 @@ export async function POST({ fetch, request }) {
   await sendEmail(fetch)(emailData);
 
   return json({
-    success: true,
-    message: `Email sent ${verificationLink}`
+    success: true
   });
 }