Merge pull request #216 from codeableorg/fix-check-email-uniqueness

refactor: simplify email validation by using verifyUniqueEmail function

Author: diegotc86

.react-router/types/+register.ts

@@ -28,4 +28,5 @@ type Params = {
   "/account/profile": {};
   "/account/orders": {};
   "/not-found": {};
+  "/verify-email": {};
 };

src/routes.ts

@@ -23,6 +23,7 @@ export default [
         route("orders", "./routes/account/orders/index.tsx"),
       ]),
       route("/not-found", "./routes/not-found/index.tsx"),
+      route("/verify-email", "./routes/verify-email/index.tsx"),
     ],
   },
 ] satisfies RouteConfig;

src/routes/verify-email/index.tsx

@@ -0,0 +1,14 @@
+import { verifyUniqueEmail } from "@/services/user.service";
+
+import type { Route } from "./+types";
+
+export async function loader({ request }: Route.ActionArgs) {
+  const url = new URL(request.url);
+  const email = url.searchParams.get("email");
+
+  if (email) {
+    const response = await verifyUniqueEmail(email);
+    return response;
+  }
+  return false;
+}

src/services/user.client-service.ts

@@ -1,10 +1,17 @@
-import { client } from "@/lib/utils";
-
-// Se mantiene para hacer la validación de correo electrónico en el registro del lado del cliente
 export async function findEmail(email: string): Promise<boolean> {
-  const body = await client<boolean>("/users/findEmail", {
-    body: { email },
-  });
+  try {
+    const response = await fetch(
+      `/verify-email?email=${encodeURIComponent(email)}`
+    );
+
+    if (!response.ok) {
+      throw new Error(`HTTP error! status: ${response.status}`);
+    }
 
-  return body;
+    const result = await response.json();
+    return result;
+  } catch (error) {
+    console.error("Error verifying email:", error);
+    return false;
+  }
 }

src/services/user.service.ts

@@ -48,3 +48,11 @@ export async function getOrCreateUser(email: string): Promise<User> {
 
   return existingUser;
 }
+
+export async function verifyUniqueEmail(email: string): Promise<boolean> {
+  const user = await prisma.user.findUnique({
+    where: { email },
+  });
+
+  return user ? false : true;
+}