updates

nextjs/src/app/CourseList.tsx

@@ -1,9 +1,9 @@
-"use client"
+"use client";
-import { useLocalCoursesQuery } from "@/hooks/localCoursesHooks";
+import { useLocalCourseNamesQuery } from "@/hooks/localCoursesHooks";
 import Link from "next/link";
 
 export default function CourseList() {
-  const { data: courses } = useLocalCoursesQuery();
+  const { data: courses } = useLocalCourseNamesQuery();
   return (
     <div>
       {courses.map((c) => (

nextjs/src/app/api/courses/[courseName]/route.ts

@@ -12,3 +12,12 @@ export async function PUT(
   await fileStorageService.saveCourseAsync(updatedCourse, previousCourse);
   return Response.json({});
 }
+
+export async function GET(
+  request: Request,
+  { params: { courseName } }: { params: { courseName: string } }
+) {
+  const courses = await fileStorageService.loadSavedCourses();
+  const course = courses.find((c) => c.settings.name === courseName);
+  return Response.json(course);
+}

nextjs/src/app/api/courses/route.ts

@@ -1,3 +1,6 @@
+import { fileStorageService } from "@/services/fileStorage/fileStorageService";
+
 export async function GET() {
-  return Response.json([]);
+  const courses = await fileStorageService.loadSavedCourses();
+  return Response.json(courses);
 }

nextjs/src/app/course/[courseName]/context/CourseContextProvider.tsx

@@ -22,50 +22,53 @@ export default function CourseContextProvider({
     DraggableItem | undefined
   >();
 
+  const updateQuiz = (day: Date) => {
+    if (!itemBeingDragged) return;
+
+    const updatedQuiz: LocalQuiz = {
+      ...(itemBeingDragged.item as LocalQuiz),
+      dueAt: dateToMarkdownString(day),
+    };
+
+    const localModule = course.modules.find((m) =>
+      m.quizzes.map((q) => q.name).includes(updatedQuiz.name)
+    );
+    if (!localModule)
+      console.log("could not find module for quiz ", updatedQuiz);
+
+    const updatedCourse: LocalCourse = {
+      ...course,
+      modules: course.modules.map((m) =>
+        m.name !== localModule?.name
+          ? m
+          : {
+              ...m,
+              quizzes: m.quizzes.map((q) =>
+                q.name === updatedQuiz.name ? updatedQuiz : q
+              ),
+            }
+      ),
+    };
+    updateCourseMutation.mutate({
+      updatedCourse,
+      previousCourse: course,
+    });
+  };
   return (
     <CourseContext.Provider
       value={{
         localCourse: course,
         startItemDrag: (d) => {
-          console.log("starting drag");
           setItemBeingDragged(d);
         },
         endItemDrag: () => {
-          console.log("stopping drag");
           setItemBeingDragged(undefined);
         },
         itemDrop: (day) => {
           console.log("dropping");
           if (itemBeingDragged && day) {
             if (itemBeingDragged.type === "quiz") {
-              const updatedQuiz: LocalQuiz = {
+              updateQuiz(day);
-                ...(itemBeingDragged.item as LocalQuiz),
-                dueAt: dateToMarkdownString(day),
-              };
-
-              const localModule = course.modules.find((m) =>
-                m.quizzes.map((q) => q.name).includes(updatedQuiz.name)
-              );
-              if (!localModule)
-                console.log("could not find module for quiz ", updatedQuiz);
-
-              const updatedCourse: LocalCourse = {
-                ...course,
-                modules: course.modules.map((m) =>
-                  m.name !== localModule?.name
-                    ? m
-                    : {
-                        ...m,
-                        quizzes: m.quizzes.map((q) =>
-                          q.name === updatedQuiz.name ? updatedQuiz : q
-                        ),
-                      }
-                ),
-              };
-              updateCourseMutation.mutate({
-                updatedCourse,
-                previousCourse: course,
-              });
             }
           }
           setItemBeingDragged(undefined);

nextjs/src/hooks/localCoursesHooks.ts

@@ -1,6 +1,5 @@
 import { LocalCourse } from "@/models/local/localCourse";
 import {
-  dataTagSymbol,
   useMutation,
   useQueryClient,
   useSuspenseQuery,
@@ -9,10 +8,10 @@ import axios from "axios";
 
 export const localCourseKeys = {
   allCourses: ["all courses"] as const,
-  courseDetail: (courseName: string) => ["all courses", courseName] as const,
+  courseDetail: (courseName: string) => ["course details", courseName] as const,
 };
 
-export const useLocalCoursesQuery = () =>
+export const useLocalCourseNamesQuery = () =>
   useSuspenseQuery({
     queryKey: localCourseKeys.allCourses,
     queryFn: async (): Promise<LocalCourse[]> => {
@@ -20,20 +19,16 @@ export const useLocalCoursesQuery = () =>
       const response = await axios.get(url);
       return response.data;
     },
+    select: (courses) => courses.map((c) => c.settings.name),
   });
 
 export const useLocalCourseDetailsQuery = (courseName: string) => {
-  const { data: courses } = useLocalCoursesQuery();
   return useSuspenseQuery({
     queryKey: localCourseKeys.courseDetail(courseName),
-    queryFn: () => {
+    queryFn: async (): Promise<LocalCourse> => {
-      const course = courses.find((c) => c.settings.name === courseName);
+      const url = `/api/courses/${courseName}`;
-      if (!course) {
+      const response = await axios.get(url);
-        console.log(courses);
+      return response.data;
-        console.log(courseName);
-        throw Error(`Could not find course with name ${courseName}`);
-      }
-      return course;
     },
   });
 };
@@ -49,10 +44,12 @@ export const useUpdateCourseMutation = (courseName: string) => {
       await axios.put(url, body);
     },
     onSuccess: () => {
-      queryClient.invalidateQueries({ queryKey: localCourseKeys.allCourses }); //optimize?
+      queryClient.invalidateQueries({
+        queryKey: localCourseKeys.courseDetail(courseName),
+      });
     },
     scope: {
-      id: "update course",
+      id: "all courses",
     },
   });
 };