{"action":"created","issue":{"url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/19","labels_url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/19/comments","events_url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/19/events","html_url":"https://github.com/michaelkourlas/voipms-sms-client/issues/19","id":92732283,"number":19,"title":"Change Pull to Refresh Behaviour","user":{"login":"bravo90078","id":12117351,"avatar_url":"https://avatars.githubusercontent.com/u/12117351?v=3","gravatar_id":"","url":"https://api.github.com/users/bravo90078","html_url":"https://github.com/bravo90078","followers_url":"https://api.github.com/users/bravo90078/followers","following_url":"https://api.github.com/users/bravo90078/following{/other_user}","gists_url":"https://api.github.com/users/bravo90078/gists{/gist_id}","starred_url":"https://api.github.com/users/bravo90078/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bravo90078/subscriptions","organizations_url":"https://api.github.com/users/bravo90078/orgs","repos_url":"https://api.github.com/users/bravo90078/repos","events_url":"https://api.github.com/users/bravo90078/events{/privacy}","received_events_url":"https://api.github.com/users/bravo90078/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-02T20:49:36Z","updated_at":"2015-07-06T04:57:51Z","closed_at":null,"body":"Since the app has been updated to only poll for messages since last received on GCM update; I suggest we alter the pull to refresh behavior to behave in the same manner and make the same call; rather than have it be a full overwriting poll. \r\n\r\nA full message sync with the server should probably only be done on first setup/connection, and if manually invoked from the settings menu. "},"comment":{"url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/comments/118718071","html_url":"https://github.com/michaelkourlas/voipms-sms-client/issues/19#issuecomment-118718071","issue_url":"https://api.github.com/repos/michaelkourlas/voipms-sms-client/issues/19","id":118718071,"user":{"login":"michaelkourlas","id":1672388,"avatar_url":"https://avatars.githubusercontent.com/u/1672388?v=3","gravatar_id":"","url":"https://api.github.com/users/michaelkourlas","html_url":"https://github.com/michaelkourlas","followers_url":"https://api.github.com/users/michaelkourlas/followers","following_url":"https://api.github.com/users/michaelkourlas/following{/other_user}","gists_url":"https://api.github.com/users/michaelkourlas/gists{/gist_id}","starred_url":"https://api.github.com/users/michaelkourlas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/michaelkourlas/subscriptions","organizations_url":"https://api.github.com/users/michaelkourlas/orgs","repos_url":"https://api.github.com/users/michaelkourlas/repos","events_url":"https://api.github.com/users/michaelkourlas/events{/privacy}","received_events_url":"https://api.github.com/users/michaelkourlas/received_events","type":"User","site_admin":false},"created_at":"2015-07-06T04:57:51Z","updated_at":"2015-07-06T04:57:51Z","body":"I've opted to go with the following approach:\r\n* When the conversation activity loads, when a GCM message is received, and when a message is sent, the app only retrieves messages dated after the most recent message retrieved.\r\n* When the user pulls to refresh, the app starts a synchronization process that is configurable in Settings. The synchronization can:\r\n  * Be limited to recent messages only, per the above.\r\n  * Retrieve deleted messages from VoIP.ms\r\n  * Propagate local deletions to VoIP.ms, and/or\r\n  * Propagate remote deletions from VoIP.ms.\r\n\r\nThis seems like a good balance, as it allows the user to decide what pull to refresh will do: do a full message sync or only retrieve the latest messages."}}