fix(Register): email collission

jkklapp · Jun 6, 2022 · 6bc23e3 · 6bc23e3
1 parent c1233f1
commit 6bc23e3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/backend/src/users/users.service.ts b/backend/src/users/users.service.ts
@@ -11,8 +11,7 @@ export class UsersService {
   }
 
   async userEmailExists(userEmail: string): Promise<boolean> {
-    return getByEmail(userEmail)
-      .then(() => true)
-      .catch(() => false);
+    const user = await getByEmail(userEmail);
+    return user !== null && user !== undefined;
   }
 }
diff --git a/frontend/src/components/Register.vue b/frontend/src/components/Register.vue
@@ -38,7 +38,9 @@
                   @blur.prevent="validate('name')"
                 />
               </div>
-              <span v-if="userNameExists" class="text-red text-xs"
+              <span
+                v-if="userNameExists"
+                class="text-red-700 dark:text-red-300 text-xs"
                 >This username already exists</span
               >
             </div>
@@ -66,6 +68,11 @@
                   @blur.prevent="validate('email')"
                 />
               </div>
+              <span
+                v-if="userEmailExists"
+                class="text-red-700 dark:text-red-300 text-xs"
+                >This email already exists</span
+              >
             </div>
             <div class="mb-6">
               <label