From ae9899dee14933c7fcc3d20b2fc362937c95f03b Mon Sep 17 00:00:00 2001 From: Mo Bitar Date: Thu, 9 Feb 2017 13:02:08 -0600 Subject: [PATCH] comment on sync function --- app/assets/javascripts/app/services/syncManager.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/assets/javascripts/app/services/syncManager.js b/app/assets/javascripts/app/services/syncManager.js index 66344b15a..d2d4b2fd3 100644 --- a/app/assets/javascripts/app/services/syncManager.js +++ b/app/assets/javascripts/app/services/syncManager.js @@ -154,6 +154,9 @@ class SyncManager { var retrieved = this.handleItemsResponse(response.retrieved_items, null); // merge only metadata for saved items + // Update 2/9/17: I just realized we may not need to handle saved_items anymore. We used to do this because we wanted to merge presentation-related metadata, + // but that has since been removed. Since this function is an important part of the functioning of the app, I'm not going to remove it just yet without careful + // testing. var omitFields = ["content", "auth_hash"]; var saved = this.handleItemsResponse(response.saved_items, omitFields);