fix: rollback to keep local WEEKLYPLANNER

InseeFrLab · Aug 20, 2024 · f610be7 · f610be7
1 parent feda624
commit f610be7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/pages/work-time/weekly-planner/WeeklyPlanner.tsx b/src/pages/work-time/weekly-planner/WeeklyPlanner.tsx
@@ -136,6 +136,7 @@ const WeeklyPlannerPage = () => {
     };
 
     // TODO: Fix the addition of "S_" prefix to the variable name (response.names)
+    // This also causes the total work time to be set to zero as lunatic-edt does not recognize the variable name
     const saveDuration = (idSurveyResponse: string, response: responsesHourChecker) => {
         const callbackData = getData(idSurvey);
         const dataCopy = { ...callbackData };

diff --git a/src/service/survey-service.ts b/src/service/survey-service.ts
@@ -551,7 +551,7 @@ const getRemoteSavedSurveyData = (
                     if (shouldSaveRemoteData(remoteSurveyData, localSurveyData)) {
                         // TEMP: WeeklyPlanner stuff (to be removed)
                         //TODO: fix a bug where other variable are overwrited if there is no weeklyPlanner
-                        const weeklyPlanner = localSurveyData?.COLLECTED?.WEEKLYPLANNER;
+                        const weeklyPlanner = localSurveyData?.COLLECTED?.WEEKLYPLANNER ?? null;
                         if (weeklyPlanner) {
                             remoteSurveyData.COLLECTED.WEEKLYPLANNER = weeklyPlanner;
                         }