From b070a7f0b6f1358ee9b3225fdd57441838ab78dd Mon Sep 17 00:00:00 2001
From: ievavold <ievavold@wisc.edu>
Date: Tue, 19 Feb 2019 14:26:01 -0600
Subject: [PATCH] fix duplicate api calls on plan load

---
 src/app/degree-planner/store/effects/plan.effects.ts | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/app/degree-planner/store/effects/plan.effects.ts b/src/app/degree-planner/store/effects/plan.effects.ts
index de75461..11daebd 100644
--- a/src/app/degree-planner/store/effects/plan.effects.ts
+++ b/src/app/degree-planner/store/effects/plan.effects.ts
@@ -411,13 +411,17 @@ const loadPlanYears = (
         ...courseTermCodes,
         ...activeTermCodes,
       ];
-      const uniqueYearCodes = unique(allTermCodes.map(toYearCode));
-      return uniqueYearCodes.sort();
+      const uniqueYearCodes = unique(allTermCodes.map(toYearCode)).sort();
+      return {
+        uniqueYearCodes,
+        notes,
+        courses,
+      };
     }),
   );
 
-  const visibleYears$ = forkJoin(uniqueYearCodes$, notesAndCourses$).pipe(
-    map(([uniqueYearCodes, { notes, courses }]) => {
+  const visibleYears$ = uniqueYearCodes$.pipe(
+    map(({ uniqueYearCodes, notes, courses }) => {
       const mapping: YearMapping = {};
       uniqueYearCodes.forEach(yearCode => {
         mapping[yearCode] = {
-- 
GitLab