Merge pull request #949 from TIP-Global-Health/issue-948

[Bug fix] Generate geo info data on first step of sync
TIP-Global-Health · Nov 22, 2023 · 55ce3c6 · 55ce3c6
2 parents 2f80733 + 404e0e9
commit 55ce3c6
Showing 1 changed file with 15 additions and 1 deletion.
diff --git a/client/src/elm/SyncManager/Update.elm b/client/src/elm/SyncManager/Update.elm
@@ -816,6 +816,9 @@ update currentDate currentTime activePage dbVersion device msg model =
 
                             modelWithSyncStatus =
                                 let
+                                    -- At general sync we get site param,
+                                    -- Theoretically, it may change, and therefore,
+                                    -- we recalculate geo info data.
                                     geoInfo =
                                         getGeoInfo syncInfoGeneral.site
                                 in
@@ -1064,14 +1067,25 @@ update currentDate currentTime activePage dbVersion device msg model =
                         noChange
 
                     else
+                        let
+                            -- This is first step of sync.
+                            -- When device is offline, we may not get any further,
+                            -- therefore, we need to generate and store geo info data here.
+                            geoInfo =
+                                getGeoInfo model.syncInfoGeneral.site
+                        in
                         update
                             currentDate
                             currentTime
                             activePage
                             dbVersion
                             device
                             (QueryIndexDb IndexDbQueryUploadPhoto)
-                            { model | syncStatus = SyncUploadPhoto errorsCount RemoteData.Loading }
+                            { model
+                                | syncStatus = SyncUploadPhoto errorsCount RemoteData.Loading
+                                , geoInfo = geoInfo
+                                , reverseGeoInfo = getReverseGeoInfo geoInfo
+                            }
 
                 _ ->
                     noChange