[PATCH v1 04/10] fix(profile): make social fields default to empty string

[Muhammad Rizki <kiizuha@gnuweeb.org>]

Previously, is optional but doesn't have default value for it. This
commit adds default value for social fields to empty string to prevent
null value.

Signed-off-by: Muhammad Rizki <kiizuha@gnuweeb.org>
---
 src/lib/schemas/profile-schema.ts             |  8 ++--
 .../(protected)/settings/profile/+page.svelte | 38 ++++++++-----------
 2 files changed, 20 insertions(+), 26 deletions(-)

diff --git a/src/lib/schemas/profile-schema.ts b/src/lib/schemas/profile-schema.ts
index c4c7228..a9981f1 100644
--- a/src/lib/schemas/profile-schema.ts
+++ b/src/lib/schemas/profile-schema.ts
@@ -13,10 +13,10 @@ export const profileSchema = z.object({
   ext_email: z.string().email("Invalid email format"),
   gender: z.enum(["m", "f"]),
   socials: z.object({
-    github_username: z.string().optional(),
-    telegram_username: z.string().optional(),
-    twitter_username: z.string().optional(),
-    discord_username: z.string().optional()
+    github_username: z.string().optional().default(""),
+    telegram_username: z.string().optional().default(""),
+    twitter_username: z.string().optional().default(""),
+    discord_username: z.string().optional().default("")
   }),
   password: z.string()
 });
diff --git a/src/routes/(protected)/settings/profile/+page.svelte b/src/routes/(protected)/settings/profile/+page.svelte
index 5ddac3b..2e9b76f 100644
--- a/src/routes/(protected)/settings/profile/+page.svelte
+++ b/src/routes/(protected)/settings/profile/+page.svelte
@@ -41,18 +41,10 @@
       formData.append("ext_email", form.data.ext_email);
       formData.append("gender", form.data.gender);
       formData.append("password", form.data.password);
-      if (form.data.socials.github_username) {
-        formData.append("socials[github_username]", form.data.socials.github_username);
-      }
-      if (form.data.socials.telegram_username) {
-        formData.append("socials[telegram_username]", form.data.socials.telegram_username);
-      }
-      if (form.data.socials.twitter_username) {
-        formData.append("socials[twitter_username]", form.data.socials.twitter_username);
-      }
-      if (form.data.socials.discord_username) {
-        formData.append("socials[discord_username]", form.data.socials.discord_username);
-      }
+      formData.append("socials[github_username]", form.data.socials.github_username);
+      formData.append("socials[telegram_username]", form.data.socials.telegram_username);
+      formData.append("socials[twitter_username]", form.data.socials.twitter_username);
+      formData.append("socials[discord_username]", form.data.socials.discord_username);
 
       const {
         data: { res },
@@ -434,17 +426,18 @@
           disabled={$submitting || !isSubmittable || isError}
           {...props}
         >
-          Update profile {isSubmittable}
+          Update profile
         </Button>
       {/snippet}
     </Dialog.Trigger>
 
     <Dialog.Content class="sm:max-w-[425px]">
-      <Dialog.Header>
-        <Dialog.Title>Update Profile Confirmation</Dialog.Title>
-        <Dialog.Description>Confirm changes to your profile here.</Dialog.Description>
-      </Dialog.Header>
-      <div>
+      <form use:enhance enctype="multipart/form-data">
+        <Dialog.Header>
+          <Dialog.Title>Update Profile Confirmation</Dialog.Title>
+          <Dialog.Description>Confirm changes to your profile here.</Dialog.Description>
+        </Dialog.Header>
+
         <Form.Field {form} name="password" class="w-full">
           <Form.Control>
             {#snippet children({ props })}
@@ -463,10 +456,11 @@
             {/snippet}
           </Form.Control>
         </Form.Field>
-      </div>
-      <Dialog.Footer>
-        <Button type="submit" onclick={handleSubmit}>Confirm</Button>
-      </Dialog.Footer>
+
+        <Dialog.Footer>
+          <Button type="submit" onclick={handleSubmit}>Confirm</Button>
+        </Dialog.Footer>
+      </form>
     </Dialog.Content>
   </form>
 </Dialog.Root>
-- 
Muhammad Rizki