removed console logs added comments

sandboxnu · Feb 2, 2025 · e9f3394 · e9f3394
1 parent 08092ed
commit e9f3394
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/packages/api/src/major/major.service.ts b/packages/api/src/major/major.service.ts
@@ -136,7 +136,6 @@ export class MajorService {
       );
       return false;
     }
-    console.log("ASDASDSA");
     return true;
   }
 

diff --git a/packages/common/src/major2-validation.ts b/packages/common/src/major2-validation.ts
@@ -287,7 +287,6 @@ export function validateMajor2(
 ): MajorValidationResult {
   const tracker = new Major2ValidationTracker(taken);
   let concentrationReq: Requirement2[] = [];
-  console.log(concentrations); // this is "undecided"
   if (major.concentrations) {
     concentrationReq = getConcentrationsRequirement(
       concentrations,
@@ -339,6 +338,7 @@ export function getConcentrationsRequirement(
   if (concentrationsRequirement.concentrationOptions.length === 0) {
     return [];
   }
+  // Allow undecided concentrations
   if (inputConcentrations === "Undecided") {
     return [];
   }

diff --git a/packages/common/src/utils.ts b/packages/common/src/utils.ts
@@ -33,6 +33,10 @@ export const majorOptionObjectComparator = (
   return majorNameComparator(a.value.toString(), b.value.toString());
 };
 
+/**
+ * Converts a list of strings or numbers into a list of option objects for the
+ * Select component, including an "Undecided" option.
+ */
 export const convertToOptionObjectsIncludingUndecided = (
   options: (string | number)[]
 ): OptionObject[] => {

diff --git a/packages/frontend/components/Plan/AddPlanModal.tsx b/packages/frontend/components/Plan/AddPlanModal.tsx
@@ -95,8 +95,6 @@ export const AddPlanModal: React.FC<AddPlanModalProps> = ({
       schedule,
     };
 
-    console.log(newPlan, " THIS IS NEW PLAN");
-
     // create the new plan
     let createdPlanId: number;
     if (isGuest) {
-Original file line number
+Diff line change
@@ Expand Up / @@ -136,7 +136,6 @@ export class MajorService { @@
           );
           return false;
         }
-        console.log("ASDASDSA");
         return true;
       }
@@ Expand Down @@