Display schedule status by summary numbers, not reason

The varianceExplanation reason can override the status on that object. We probably don't want that here.
SeedCompany · Oct 16, 2024 · 7298a4c · 7298a4c
1 parent f5a35cf
commit 7298a4c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/scenes/Dashboard/ProgressReportsWidget/ProgressReportsGrid.tsx b/src/scenes/Dashboard/ProgressReportsWidget/ProgressReportsGrid.tsx
@@ -97,7 +97,7 @@ export const ProgressReportsColumnMap = {
     ...enumColumn(ScheduleStatusList, ScheduleStatusLabels, {
       orderByIndex: true,
     }),
-    valueGetter: (_, row) => row.varianceExplanation.scheduleStatus,
+    valueGetter: (_, row) => row.cumulativeSummary?.scheduleStatus,
   },
   cumulativeSummary: {
     headerName: 'Cumulative Progress',

diff --git a/src/scenes/Dashboard/ProgressReportsWidget/progressReportsDataGridRow.graphql b/src/scenes/Dashboard/ProgressReportsWidget/progressReportsDataGridRow.graphql
@@ -34,7 +34,6 @@ fragment progressReportsDataGridRow on ProgressReport {
     value
   }
   varianceExplanation {
-    scheduleStatus
     reasons {
       value
     }
@@ -43,6 +42,7 @@ fragment progressReportsDataGridRow on ProgressReport {
     planned
     actual
     variance
+    scheduleStatus
   }
   teamNews {
     items {