fixed comments

Dsek-LTH · Apr 16, 2024 · 8922105 · 8922105
2 parents b3cfe0f + 8de7c29
commit 8922105
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/src/lib/server/keycloak/index.ts b/src/lib/server/keycloak/index.ts
@@ -186,7 +186,7 @@ async function hasEmail(email: string) {
 }
 
 async function getEmail(username: string) {
-  if (!enabled) return false;
+  if (!enabled) return;
   const client = await connect();
   const user = await client.users.find({ username });
   if (user.length === 1) return user[0]?.email;

diff --git a/src/routes/(app)/members/[studentId]/+page.server.ts b/src/routes/(app)/members/[studentId]/+page.server.ts
@@ -67,10 +67,10 @@ export const load: PageServerLoad = async ({ locals, params }) => {
     member.id,
   );
 
-  let email;
-  if (member.studentId !== null) {
-    email = await keycloak.getEmail(member.studentId);
-  }
+  const email =
+    member.studentId !== null
+      ? await keycloak.getEmail(member.studentId)
+      : undefined;
 
   return {
     form: await superValidate(member, memberSchema),

diff --git a/vercel.sh b/vercel.sh
@@ -1,7 +1,6 @@
 #!/bin/bash
 
-if test "$VERCEL_ENV" = "production"
-then 
+if [[ $VERCEL_ENV == "production"  ]] ; then 
   pnpm run build && pnpm prisma migrate deploy
 else 
   # prod and preview use the same database.