updated addplan modal to support empty plan name

sandboxnu · Jan 28, 2025 · 65ca248 · 65ca248
1 parent d1cfe06
commit 65ca248
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/packages/frontend/components/Plan/AddPlanModal.tsx b/packages/frontend/components/Plan/AddPlanModal.tsx
@@ -60,6 +60,13 @@ export const AddPlanModal: React.FC<AddPlanModalProps> = ({
   const { onOpen, onClose: onCloseDisplay, isOpen } = useDisclosure();
   const { supportedMajorsData, error: supportedMajorsError } =
     useSupportedMajors();
+
+  // Generate default plan title using formatted date and time
+  const generateDefaultPlanTitle = () => {
+    const now = new Date();
+    return `Plan ${now.toLocaleDateString()} ${now.toLocaleTimeString()}`;
+  };
+
   const {
     register,
     handleSubmit,
@@ -87,7 +94,7 @@ export const AddPlanModal: React.FC<AddPlanModalProps> = ({
   const onSubmitHandler = async (payload: CreatePlanDtoWithoutSchedule) => {
     const schedule = createEmptySchedule();
     const newPlan: CreatePlanDto = {
-      name: payload.name,
+      name: payload.name || generateDefaultPlanTitle(),
       catalogYear: isNoMajorSelected ? undefined : payload.catalogYear,
       major: isNoMajorSelected ? undefined : payload.major,
       concentration: isNoMajorSelected ? undefined : payload.concentration,
@@ -161,13 +168,12 @@ export const AddPlanModal: React.FC<AddPlanModalProps> = ({
     yearSupportedMajors?.[majorName ?? ""]?.verified ?? false;
 
   const isValidForm =
-    (title &&
-      catalogYear &&
+    (catalogYear &&
       majorName &&
       (!isConcentrationRequired || concentration) &&
       (!isValidatedMajor ? agreeToBetaMajor : true)) ||
     // Valid plan for no major selected
-    (title && isNoMajorSelected);
+    isNoMajorSelected;
 
   const noMajorHelperLabel = (
     <Stack>
@@ -227,10 +233,9 @@ export const AddPlanModal: React.FC<AddPlanModalProps> = ({
                   label="Title"
                   id="name"
                   type="text"
-                  placeholder="My Plan"
+                  placeholder={generateDefaultPlanTitle()}
                   error={errors.name}
                   {...register("name", {
-                    required: "Title is required",
                     pattern: noLeadOrTrailWhitespacePattern,
                   })}
                 />