Merge pull request #3 from Codeunia/fix/auth

feat(auth): improve sign-in flow with error handling and success response

Author: akshay0611

app/(Root)/page.tsx

@@ -1,14 +1,20 @@
-// import {Image} from "next/image"
 import Link from "next/link";
 import Image from "next/image";
 
 import React from 'react'
 import {Button} from '@/components/ui/button'
+import InterviewCard from "@/components/interviewCard";
+import { Interview } from "@/types";
 // import { Link } from 'lucide-react'
 
 
 
 const page = () => {
+  const userInterviews: Interview[] = [];
+  const allInterview: Interview[] = [];
+  const user = { id: "1" };
+  const hasPastInterviews = userInterviews.length > 0;
+  const hasUpcomingInterviews = allInterview.length > 0;
   return (
    <>
    <section className="card-cta">
@@ -37,7 +43,7 @@ const page = () => {
 
         <div className="interviews-section">
           {hasPastInterviews ? (
-            userInterviews?.map((interview) => (
+            userInterviews?.map((interview: Interview) => (
               <InterviewCard
                 key={interview.id}
                 userId={user?.id}
@@ -59,7 +65,7 @@ const page = () => {
 
         <div className="interviews-section">
           {hasUpcomingInterviews ? (
-            allInterview?.map((interview) => (
+            allInterview?.map((interview: Interview) => (
               <InterviewCard
                 key={interview.id}
                 userId={user?.id}

components/AuthForm.tsx

@@ -82,11 +82,16 @@ const AuthForm = ({ type }: { type: FormType }) => {
           return;
         }
 
-        await signIn({
+        const result = await signIn({
           email,
           idToken,
         });
 
+        if (!result.success) {
+          toast.error(result.message);
+          return;
+        }
+
         toast.success("Signed in successfully.");
         router.push("/");
       }

components/interviewCard.tsx

@@ -6,7 +6,7 @@ import { Button } from "./ui/button";
 import DisplayTechIcons from "./DisplayTechIcons";
 
 import { cn, getRandomInterviewCover } from "@/lib/utils";
-import { getFeedbackByInterviewId } from "@lib/actions/general.action";
+import { getFeedbackByInterviewId } from "@/lib/actions/general.action";
 import { InterviewCardProps } from "@/types";
 // @/lib/actions/general.action
 

lib/actions/auth.action.ts

@@ -83,9 +83,13 @@ export async function signIn(params: SignInParams) {
       };
 
     await setSessionCookie(idToken);
+
+    return {
+      success: true,
+    };
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   } catch (error: any) {
-    console.log("");
+    console.log(error);
 
     return {
       success: false,

lib/actions/general.action.ts

@@ -0,0 +1,38 @@
+"use server";
+
+import { db } from "@/firebase/admin";
+import { Feedback, GetFeedbackByInterviewIdParams } from "@/types";
+
+export async function getFeedbackByInterviewId(
+  params: GetFeedbackByInterviewIdParams
+): Promise<Feedback | null> {
+  try {
+    const { userId, interviewId } = params;
+
+    const feedbackRef = await db
+      .collection("users")
+      .doc(userId)
+      .collection("interviews")
+      .doc(interviewId)
+      .collection("feedbacks")
+      .get();
+
+    if (feedbackRef.empty) return null;
+
+    const feedback = feedbackRef.docs[0].data();
+
+    return {
+      id: feedbackRef.docs[0].id,
+      interviewId: feedback.interviewId,
+      totalScore: feedback.totalScore,
+      categoryScores: feedback.categoryScores,
+      strengths: feedback.strengths,
+      areasForImprovement: feedback.areasForImprovement,
+      finalAssessment: feedback.finalAssessment,
+      createdAt: feedback.createdAt.toDate(),
+    };
+  } catch (error) {
+    console.log(error);
+    return null;
+  }
+}