From 9dc869552e2948c0e90b75e9feab6d7e77aa5cfb Mon Sep 17 00:00:00 2001
From: Advay Anand <advay.anand@gmail.com>
Date: Thu, 2 Mar 2023 23:40:22 -0800
Subject: [PATCH 1/2] add reviews page and show users reviews

---
 api/src/controllers/reviews.ts                | 14 ++++-
 site/src/App.tsx                              |  2 +
 site/src/component/SideBar/SideBar.tsx        | 12 +++++
 .../component/UserReviews/UserReviews.scss    | 17 +++++++
 .../src/component/UserReviews/UserReviews.tsx | 51 +++++++++++++++++++
 site/src/pages/ReviewsPage/ReviewsPage.scss   |  0
 site/src/pages/ReviewsPage/index.tsx          | 41 +++++++++++++++
 7 files changed, 135 insertions(+), 2 deletions(-)
 create mode 100644 site/src/component/UserReviews/UserReviews.scss
 create mode 100644 site/src/component/UserReviews/UserReviews.tsx
 create mode 100644 site/src/pages/ReviewsPage/ReviewsPage.scss
 create mode 100644 site/src/pages/ReviewsPage/index.tsx

diff --git a/api/src/controllers/reviews.ts b/api/src/controllers/reviews.ts
index 40e88a2d..84b2599f 100644
--- a/api/src/controllers/reviews.ts
+++ b/api/src/controllers/reviews.ts
@@ -165,7 +165,17 @@ router.post("/", async function (req, res, next) {
  * Delete a review
  */
 router.delete('/', async (req, res, next) => {
-  if (req.session.passport?.admin) {
+
+  const checkUser = async () => {
+
+    let review = await getDocuments(COLLECTION_NAMES.REVIEWS, {
+      _id: new ObjectID(req.body.id)
+    });
+
+    return review.length > 0 && review[0].userID === req.session.passport?.user.id;
+  }
+
+  if (req.session.passport?.admin || await checkUser()) {
     console.log(`Deleting review ${req.body.id}`);
 
     let status = await deleteDocument(COLLECTION_NAMES.REVIEWS, {
@@ -179,7 +189,7 @@ router.delete('/', async (req, res, next) => {
     res.json(status);
   }
   else {
-    res.json({ error: 'Must be an admin to delete reviews!' });
+    res.json({ error: 'Must be an admin or review author to delete reviews!' });
   }
 })
 
diff --git a/site/src/App.tsx b/site/src/App.tsx
index 779ba6a0..a89e88f4 100644
--- a/site/src/App.tsx
+++ b/site/src/App.tsx
@@ -19,6 +19,7 @@ import ErrorPage from './pages/ErrorPage';
 import RoadmapPage from './pages/RoadmapPage';
 import ZotisticsPage from './pages/ZotisticsPage';
 import AdminPage from './pages/AdminPage';
+import ReviewsPage from './pages/ReviewsPage';
 import SideBar from './component/SideBar/SideBar';
 
 import { useAppSelector } from './store/hooks';
@@ -51,6 +52,7 @@ export default function App() {
             <Route path='/course/:id+' component={CoursePage} />
             <Route path='/professor/:id' component={ProfessorPage} />
             <Route path='/admin' component={AdminPage} />
+            <Route path='/reviews'  component={ReviewsPage} />
             <Route component={ErrorPage} />
           </Switch>
           <Footer />
diff --git a/site/src/component/SideBar/SideBar.tsx b/site/src/component/SideBar/SideBar.tsx
index 88220ed3..64c25f39 100644
--- a/site/src/component/SideBar/SideBar.tsx
+++ b/site/src/component/SideBar/SideBar.tsx
@@ -67,6 +67,18 @@ const SideBar: FC = ({ children }) => {
           </span>
         </NavLink>
       </li>
+      {isLoggedIn && 
+      <li>
+        <NavLink to='/reviews' activeClassName='sidebar-active' onClick={closeSidebar}>
+          <div>
+            <Icon name='sticky note outline' size='large' />
+          </div>
+          <span>
+            Reviews
+          </span>
+        </NavLink>
+      </li>
+      }
       {isAdmin && <>
       <li>
         <NavLink to='/admin/verify' activeClassName='sidebar-active' onClick={closeSidebar}>
diff --git a/site/src/component/UserReviews/UserReviews.scss b/site/src/component/UserReviews/UserReviews.scss
new file mode 100644
index 00000000..f02a2e5d
--- /dev/null
+++ b/site/src/component/UserReviews/UserReviews.scss
@@ -0,0 +1,17 @@
+.user-reviews-container {
+    padding: 2rem;
+    background-color: white;
+    border-radius: var(--border-radius);
+}
+
+.user-reviews {
+    background-color: white;
+    border-radius: var(--border-radius);
+    margin-bottom: 3vh;
+
+    .user-reviews-footer {
+        flex-direction: row;
+        width: 100%;
+        align-items: flex-end;
+    }
+}
\ No newline at end of file
diff --git a/site/src/component/UserReviews/UserReviews.tsx b/site/src/component/UserReviews/UserReviews.tsx
new file mode 100644
index 00000000..812d7a8f
--- /dev/null
+++ b/site/src/component/UserReviews/UserReviews.tsx
@@ -0,0 +1,51 @@
+import axios, { AxiosResponse } from "axios";
+import React, { FC, useEffect, useState } from "react";
+import SubReview from '../../component/Review/SubReview';
+import Button from 'react-bootstrap/Button';
+import { Divider } from 'semantic-ui-react';
+import { ReviewData } from "src/types/types";
+import './UserReviews.scss';
+import { useCookies } from "react-cookie";
+
+const UserReviews: FC = () => {
+    const [reviews, setReviews] = useState<ReviewData[]>([]);
+    const [loaded, setLoaded] = useState<boolean>(false);
+    const [cookies, setCookie] = useCookies(['user']);
+
+    const getUserReviews = async () => {
+        const response: AxiosResponse<ReviewData[]> = await axios.get(`/reviews?userID=${cookies.user.id}`);
+        setReviews(response.data);
+        setLoaded(true);
+    }
+
+    useEffect(() => {
+        getUserReviews();
+    }, []);
+
+    const deleteReview = async (reviewID: string) => {
+        await axios.delete('/reviews', { data: { id: reviewID } });
+        setReviews(reviews.filter(review => review._id !== reviewID));
+    }
+
+    if (!loaded) {
+        return <p>Loading...</p>;
+    } else if (reviews.length === 0) {
+        return <p>No reviews to display at the moment.</p>;
+    } else {
+        return <div className='user-reviews-container'>
+            <h1>Your Reviews</h1>
+            <p>Deleting a review will remove it permanently.</p>
+            {
+                reviews.map((review, i) => <div key={`user-reviews-${i}`} className='user-reviews'>
+                    <Divider />
+                    <SubReview review={review}></SubReview>
+                    <div className='user-reviews-footer'>
+                        <Button variant='danger' className='mr-3' onClick={() => deleteReview(review._id!)}>Delete</Button>
+                    </div>
+                </div>)
+            }
+        </div>
+    }
+};
+
+export default UserReviews;
\ No newline at end of file
diff --git a/site/src/pages/ReviewsPage/ReviewsPage.scss b/site/src/pages/ReviewsPage/ReviewsPage.scss
new file mode 100644
index 00000000..e69de29b
diff --git a/site/src/pages/ReviewsPage/index.tsx b/site/src/pages/ReviewsPage/index.tsx
new file mode 100644
index 00000000..fa70b038
--- /dev/null
+++ b/site/src/pages/ReviewsPage/index.tsx
@@ -0,0 +1,41 @@
+import React, { FC, useEffect, useState } from "react";
+import UserReviews from '../../component/UserReviews/UserReviews';
+import Error from '../../component/Error/Error';
+import './ReviewsPage.scss';
+import { useLocation } from "react-router-dom";
+import { useCookies } from "react-cookie";
+
+const ReviewsPage: FC = () => {
+    const location = useLocation();
+    const [loaded, setLoaded] = useState<boolean>(false);
+    const [cookies, setCookie] = useCookies(['user']);
+    const [authorized, setAuthorized] = useState<boolean>(false);
+
+    // user has to be logged in to view this page
+    const checkLoggedIn = async () => {
+        const loggedIn: boolean = cookies.hasOwnProperty('user');
+        setAuthorized(loggedIn);
+        setLoaded(true);
+    }
+
+    useEffect(() => {
+        checkLoggedIn();
+    }, []);
+
+    if (!loaded) {
+        return <p>Loading...</p>;
+    }
+    else if (!authorized) {
+        return (
+            <Error message='Access Denied: Log in to view this page.'></Error>
+        );
+    }
+    else {
+        if (location.pathname.includes('reviews')) {
+            return <UserReviews />
+        }
+    }
+    return <Error message='Invalid Reviews Page'></Error>
+};
+
+export default ReviewsPage;
\ No newline at end of file

From 830e4b092292c00062fa8d8573084ea3ca1f721d Mon Sep 17 00:00:00 2001
From: Advay Anand <advay.anand@gmail.com>
Date: Fri, 9 Jun 2023 00:58:01 -0700
Subject: [PATCH 2/2] add /api before api calls

---
 site/src/component/UserReviews/UserReviews.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/site/src/component/UserReviews/UserReviews.tsx b/site/src/component/UserReviews/UserReviews.tsx
index 812d7a8f..8dae070a 100644
--- a/site/src/component/UserReviews/UserReviews.tsx
+++ b/site/src/component/UserReviews/UserReviews.tsx
@@ -13,7 +13,7 @@ const UserReviews: FC = () => {
     const [cookies, setCookie] = useCookies(['user']);
 
     const getUserReviews = async () => {
-        const response: AxiosResponse<ReviewData[]> = await axios.get(`/reviews?userID=${cookies.user.id}`);
+        const response: AxiosResponse<ReviewData[]> = await axios.get(`/api/reviews?userID=${cookies.user.id}`);
         setReviews(response.data);
         setLoaded(true);
     }
@@ -23,7 +23,7 @@ const UserReviews: FC = () => {
     }, []);
 
     const deleteReview = async (reviewID: string) => {
-        await axios.delete('/reviews', { data: { id: reviewID } });
+        await axios.delete('/api/reviews', { data: { id: reviewID } });
         setReviews(reviews.filter(review => review._id !== reviewID));
     }