now works when logged out, yet to test when logged in

cornell-dti · Sep 26, 2024 · ef3e1d5 · ef3e1d5
1 parent 6502c60
commit ef3e1d5
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/client/src/modules/Course/Components/Course.tsx b/client/src/modules/Course/Components/Course.tsx
@@ -186,9 +186,7 @@ export const Course = () => {
     })
     Session.setPersistent({ review_num: selectedClass?.classNum })
     Session.setPersistent({ courseId: selectedClass?._id })
-
-    signIn('profile')
-  }
+   }
 
   /**
    * Clear review stored in session storage

diff --git a/client/src/modules/Course/Components/ReviewModal.tsx b/client/src/modules/Course/Components/ReviewModal.tsx
@@ -59,7 +59,7 @@ const ReviewModal = ({
   const [anonymousOpen, setAnonymousOpen] = useState<boolean>(false)
   const [noReviews, setNoReviews] = useState<boolean>(false)
 
-  const {isLoggedIn, netId} = useAuthOptionalLogin()
+  const {isLoggedIn, netId, signIn} = useAuthOptionalLogin()
 
   const [valid, setValid] = useState<Valid>({
     professor: false,
@@ -145,9 +145,11 @@ const ReviewModal = ({
 
   // Handle click of submit button
   function onSubmitReview() {
-    if (!noReviews) {
+    if (!noReviews && isLoggedIn) {
       handleSubmitReview()
+      signIn('profile')
     } else {
+      handleSubmitReview()
       setAnonymousOpen(true)
       setOpen(false)
     }

diff --git a/client/src/modules/Profile/Component/Profile.tsx b/client/src/modules/Profile/Component/Profile.tsx
@@ -141,6 +141,7 @@ const Profile = () => {
 
         clearSessionReview()
         if (response.status === 200) {
+          console.log('toast!')
           toast.success(
             'Thanks for reviewing! New reviews are updated every 24 hours.'
           )
@@ -184,6 +185,15 @@ const Profile = () => {
     return (
       <div className={styles.page}>
         <Navbar userInput="" />
+        <ToastContainer
+          position="top-right"
+          autoClose={5000}
+          hideProgressBar={false}
+          closeOnClick
+          pauseOnFocusLoss
+          draggable
+          pauseOnHover
+        />
         <div className={styles.container}>
           <div className={styles.usersection}>
             <UserInfo