{"action":"created","issue":{"url":"https://api.github.com/repos/projectbuendia/client/issues/171","labels_url":"https://api.github.com/repos/projectbuendia/client/issues/171/labels{/name}","comments_url":"https://api.github.com/repos/projectbuendia/client/issues/171/comments","events_url":"https://api.github.com/repos/projectbuendia/client/issues/171/events","html_url":"https://github.com/projectbuendia/client/pull/171","id":120331065,"number":171,"title":"Removing old snackbar.","user":{"login":"llvasconcellos","id":1710823,"avatar_url":"https://avatars.githubusercontent.com/u/1710823?v=3","gravatar_id":"","url":"https://api.github.com/users/llvasconcellos","html_url":"https://github.com/llvasconcellos","followers_url":"https://api.github.com/users/llvasconcellos/followers","following_url":"https://api.github.com/users/llvasconcellos/following{/other_user}","gists_url":"https://api.github.com/users/llvasconcellos/gists{/gist_id}","starred_url":"https://api.github.com/users/llvasconcellos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/llvasconcellos/subscriptions","organizations_url":"https://api.github.com/users/llvasconcellos/orgs","repos_url":"https://api.github.com/users/llvasconcellos/repos","events_url":"https://api.github.com/users/llvasconcellos/events{/privacy}","received_events_url":"https://api.github.com/users/llvasconcellos/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-12-04T04:34:01Z","updated_at":"2015-12-04T07:14:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/projectbuendia/client/pulls/171","html_url":"https://github.com/projectbuendia/client/pull/171","diff_url":"https://github.com/projectbuendia/client/pull/171.diff","patch_url":"https://github.com/projectbuendia/client/pull/171.patch"},"body":"*** DON'T MERGE ***\r\nMigration of error messages to new snackbar and removing the old one.\r\n\r\nI still need to test and migrate the UpdateNotificationUi messages.\r\nThis is interesting to see the messages pilling up.\r\nI noticed that some messages are repeating. That's because they are getting triggered again and this was the previous behavior, we just didn't notice because new messages overlapped the older ones.\r\nWe need to think on a mechanism to avoid this. This could be done on the SnackBar level (to not allow duplicates) or can be done where the messages are generated. I accept suggestions.\r\n\r\n![device-2015-12-04-023341](https://cloud.githubusercontent.com/assets/1710823/11581944/76f3f544-9a2f-11e5-8ab7-96b1df01c8dc.png)\r\n"},"comment":{"url":"https://api.github.com/repos/projectbuendia/client/issues/comments/161896068","html_url":"https://github.com/projectbuendia/client/pull/171#issuecomment-161896068","issue_url":"https://api.github.com/repos/projectbuendia/client/issues/171","id":161896068,"user":{"login":"capnfabs","id":1336508,"avatar_url":"https://avatars.githubusercontent.com/u/1336508?v=3","gravatar_id":"","url":"https://api.github.com/users/capnfabs","html_url":"https://github.com/capnfabs","followers_url":"https://api.github.com/users/capnfabs/followers","following_url":"https://api.github.com/users/capnfabs/following{/other_user}","gists_url":"https://api.github.com/users/capnfabs/gists{/gist_id}","starred_url":"https://api.github.com/users/capnfabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/capnfabs/subscriptions","organizations_url":"https://api.github.com/users/capnfabs/orgs","repos_url":"https://api.github.com/users/capnfabs/repos","events_url":"https://api.github.com/users/capnfabs/events{/privacy}","received_events_url":"https://api.github.com/users/capnfabs/received_events","type":"User","site_admin":false},"created_at":"2015-12-04T07:14:31Z","updated_at":"2015-12-04T07:14:31Z","body":"I think I would do it at the snackbar level similarly to how [Android does notifications](http://developer.android.com/guide/topics/ui/notifiers/notifications.html#Updating). For notifications, you have to pass a unique numeric ID to the notification function, which defines whether it overwrites an existing notification or not. For snackbar, this means you'd have to re-architect some of the functions to accept an ID instead of generate an ID, but I think this is a pretty clean solution."}}