Merge pull request #24467 from Expensify/yuwen-backToPolicyList

Revert back to using policyIDList instead of the last modified map when reconnecting the App
Expensify · Aug 14, 2023 · 21d483c · 21d483c
2 parents 6a5b1a4 + 7be2059
commit 21d483c
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/libs/actions/App.js b/src/libs/actions/App.js
@@ -53,13 +53,13 @@ function confirmReadyToOpenApp() {
 
 /**
  * @param {Array} policies
- * @return {Object} map of policy id to lastUpdated
+ * @return {Array<String>} array of policy ids
  */
-function getNonOptimisticPolicyIDToLastModifiedMap(policies) {
+function getNonOptimisticPolicyIDs(policies) {
     return _.chain(policies)
-        .reject((policy) => lodashGet(policy, 'pendingAction', '') === CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD)
-        .map((policy) => [policy.id, policy.lastModified || 0])
-        .object()
+        .reject((policy) => lodashGet(policy, 'pendingAction', null) === CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD)
+        .pluck('id')
+        .compact()
         .value();
 }
 
@@ -133,7 +133,7 @@ function getPolicyParamsForOpenOrReconnect() {
                 waitForCollectionCallback: true,
                 callback: (policies) => {
                     Onyx.disconnect(connectionID);
-                    resolve({policyIDToLastModified: JSON.stringify(getNonOptimisticPolicyIDToLastModifiedMap(policies))});
+                    resolve({policyIDList: getNonOptimisticPolicyIDs(policies)});
                 },
             });
         });