capped years at 5

sandboxnu · Nov 17, 2024 · 491824b · 491824b
1 parent 0734264
commit 491824b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/packages/frontend/components/Plan/AddYearButton.tsx b/packages/frontend/components/Plan/AddYearButton.tsx
@@ -17,7 +17,11 @@ export const AddYearButton: React.FC<AddYearButton> = ({
     mutateStudentWithUpdatedPlan(updatedPlan);
   };
   return (
-    <BlueButton onClick={addYear} leftIcon={<AddIcon />}>
+    <BlueButton
+      onClick={addYear}
+      leftIcon={<AddIcon />}
+      isDisabled={plan.schedule.years.length >= 5}
+    >
       Add Year
     </BlueButton>
   );

diff --git a/packages/frontend/utils/plan/addEmptyDndYearToPlan.ts b/packages/frontend/utils/plan/addEmptyDndYearToPlan.ts
@@ -7,6 +7,7 @@ import { getCourseCount, prepareYearForDnd } from "./preparePlanForDnd";
 export const addEmptyDndYearToPlan = (
   plan: PlanModel<string>
 ): PlanModel<string> => {
+  // Max 5 years
   const updatedPlan = produce(plan, (draftPlan) => {
     const schedule = draftPlan.schedule;
     const newYearNum = schedule.years.length + 1;