delete item experience

nextjs/src/app/course/[courseName]/modules/[moduleName]/assignment/[assignmentName]/AssignmentButtons.tsx

@@ -135,15 +135,20 @@ export function AssignmentButtons({
                         moduleName,
                         itemName: assignmentName,
                       });
-
                       router.refresh();
-                      setIsLoading(false);
+                      // setIsLoading(false); //refreshing the router will make spinner go away
                     }}
+                    disabled={deleteLocal.isPending || isLoading}
                     className="btn-danger"
                   >
                     Yes
                   </button>
-                  <button onClick={closeModal}>No</button>
+                  <button
+                    onClick={closeModal}
+                    disabled={deleteLocal.isPending || isLoading}
+                  >
+                    No
+                  </button>
                 </div>
                 {(deleteLocal.isPending || isLoading) && <Spinner />}
               </div>

nextjs/src/app/course/[courseName]/modules/[moduleName]/assignment/[assignmentName]/loading.tsx

@@ -0,0 +1,10 @@
+import { Spinner } from "@/components/Spinner";
+import React from "react";
+
+export default function Loading() {
+  return (
+    <div>
+      <Spinner />
+    </div>
+  );
+}

nextjs/src/hooks/localCourse/courseItemHooks.ts

@@ -261,11 +261,11 @@ export const useDeleteItemMutation = <T extends CourseItemType>(type: T) => {
       await axiosClient.delete(url);
     },
     onSuccess: async (_, { moduleName, itemName }) => {
-      queryClient.invalidateQueries({
+      await queryClient.invalidateQueries({
         queryKey: localCourseKeys.allItemsOfType(courseName, moduleName, type),
         // refetchType: "all",
       });
-      queryClient.invalidateQueries({
+      await queryClient.invalidateQueries({
         queryKey: localCourseKeys.itemOfType(
           courseName,
           moduleName,

nextjs/src/services/fileStorage/settingsFileStorageService.ts

@@ -38,7 +38,7 @@ const populateDefaultValues = (settingsFromFile: LocalCourseSettings) => {
     AssignmentSubmissionType.ONLINE_TEXT_ENTRY,
     AssignmentSubmissionType.ONLINE_UPLOAD,
   ];
-  const defaultFileUploadTypes = ["pdf", "jpg", "jpeg", "pdf"];
+  const defaultFileUploadTypes = ["pdf", "jpg", "jpeg"];
 
   const settings: LocalCourseSettings = {
     ...settingsFromFile,