Address review comments

Author: jhodapp

src/middleware.ts

@@ -19,18 +19,18 @@ export default async function middleware(req: NextRequest) {
 
   // 3. Get the session from the cookie
   const sessionCookie = req.cookies.get("id");
-  const validSession = sessionCookie?.value;
+  const isValidSession = sessionCookie?.value;
 
   // 4. Redirect to / if the user is not authenticated
   // 4b. TODO: Check session validity/expiration?
-  if (isProtectedRoute && !validSession) {
+  if (isProtectedRoute && !isValidSession) {
     return NextResponse.redirect(new URL("/", req.nextUrl));
   }
 
   // 5. Redirect to /dashboard if the user is authenticated
   if (
     isPublicRoute &&
-    validSession &&
+    isValidSession &&
     !req.nextUrl.pathname.startsWith("/dashboard")
   ) {
     return NextResponse.redirect(new URL("/dashboard", req.nextUrl));

src/types/coaching-session.ts

@@ -69,26 +69,6 @@ export function filterAndSortCoachingSessions(
   return filteredSessions;
 }
 
-// export function sortCoachingSessionArray(
-//   sessions: CoachingSession[],
-//   order: SortOrder
-// ): CoachingSession[] {
-//   if (order == SortOrder.Ascending) {
-//     sessions.sort(
-//       (a, b) =>
-//         new Date(a.date.toString()).getTime() -
-//         new Date(b.date.toString()).getTime()
-//     );
-//   } else if (order == SortOrder.Descending) {
-//     sessions.sort(
-//       (a, b) =>
-//         new Date(b.date.toString()).getTime() -
-//         new Date(a.date.toString()).getTime()
-//     );
-//   }
-//   return sessions;
-// }
-
 export function getCoachingSessionById(
   id: string,
   sessions: CoachingSession[]