{"action":"created","issue":{"url":"https://api.github.com/repos/Telaxus/EPESI/issues/32","labels_url":"https://api.github.com/repos/Telaxus/EPESI/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/Telaxus/EPESI/issues/32/comments","events_url":"https://api.github.com/repos/Telaxus/EPESI/issues/32/events","html_url":"https://github.com/Telaxus/EPESI/pull/32","id":86339432,"number":32,"title":"Base/Notify v2.0:","user":{"login":"georgehristov","id":8128250,"avatar_url":"https://avatars.githubusercontent.com/u/8128250?v=3","gravatar_id":"","url":"https://api.github.com/users/georgehristov","html_url":"https://github.com/georgehristov","followers_url":"https://api.github.com/users/georgehristov/followers","following_url":"https://api.github.com/users/georgehristov/following{/other_user}","gists_url":"https://api.github.com/users/georgehristov/gists{/gist_id}","starred_url":"https://api.github.com/users/georgehristov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/georgehristov/subscriptions","organizations_url":"https://api.github.com/users/georgehristov/orgs","repos_url":"https://api.github.com/users/georgehristov/repos","events_url":"https://api.github.com/users/georgehristov/events{/privacy}","received_events_url":"https://api.github.com/users/georgehristov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-06-08T21:34:05Z","updated_at":"2015-06-11T12:11:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Telaxus/EPESI/pulls/32","html_url":"https://github.com/Telaxus/EPESI/pull/32","diff_url":"https://github.com/Telaxus/EPESI/pull/32.diff","patch_url":"https://github.com/Telaxus/EPESI/pull/32.patch"},"body":"- use `READ_ONLY_SESSION` when retrieving notifications to avoid locking the session\r\n- keep the last_refresh_time in the js variable\r\n- rewritten js script file for clarity"},"comment":{"url":"https://api.github.com/repos/Telaxus/EPESI/issues/comments/111110447","html_url":"https://github.com/Telaxus/EPESI/pull/32#issuecomment-111110447","issue_url":"https://api.github.com/repos/Telaxus/EPESI/issues/32","id":111110447,"user":{"login":"bukowskiadam","id":1633333,"avatar_url":"https://avatars.githubusercontent.com/u/1633333?v=3","gravatar_id":"","url":"https://api.github.com/users/bukowskiadam","html_url":"https://github.com/bukowskiadam","followers_url":"https://api.github.com/users/bukowskiadam/followers","following_url":"https://api.github.com/users/bukowskiadam/following{/other_user}","gists_url":"https://api.github.com/users/bukowskiadam/gists{/gist_id}","starred_url":"https://api.github.com/users/bukowskiadam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bukowskiadam/subscriptions","organizations_url":"https://api.github.com/users/bukowskiadam/orgs","repos_url":"https://api.github.com/users/bukowskiadam/repos","events_url":"https://api.github.com/users/bukowskiadam/events{/privacy}","received_events_url":"https://api.github.com/users/bukowskiadam/received_events","type":"User","site_admin":false},"created_at":"2015-06-11T12:11:22Z","updated_at":"2015-06-11T12:11:22Z","body":"I've created my solution basing on yours new. It's [here](/Telaxus/EPESI/tree/new_notify)\r\n\r\nI've got a question about `message_refresh_limit` - why we should limit messages?\r\nNow, if I properly get it, it works like that:\r\n1. Check for new notifications - 5 new\r\n2. Retrieve only 3 and show them\r\n3. Wait for the next refresh\r\n4. Retrieve next 2 and show them\r\n5. Update last refresh time.\r\nIs there a need for that? Chrome for instance will show only 4 active notifications, next ones will appear only if you'll dismiss active. I've removed this, but we can revert it."}}