{"action":"created","issue":{"url":"https://api.github.com/repos/novoda/download-manager/issues/78","labels_url":"https://api.github.com/repos/novoda/download-manager/issues/78/labels{/name}","comments_url":"https://api.github.com/repos/novoda/download-manager/issues/78/comments","events_url":"https://api.github.com/repos/novoda/download-manager/issues/78/events","html_url":"https://github.com/novoda/download-manager/pull/78","id":93778119,"number":78,"title":"Less isAllowToDownload callbacks","user":{"login":"ouchadam","id":1848238,"avatar_url":"https://avatars.githubusercontent.com/u/1848238?v=3","gravatar_id":"","url":"https://api.github.com/users/ouchadam","html_url":"https://github.com/ouchadam","followers_url":"https://api.github.com/users/ouchadam/followers","following_url":"https://api.github.com/users/ouchadam/following{/other_user}","gists_url":"https://api.github.com/users/ouchadam/gists{/gist_id}","starred_url":"https://api.github.com/users/ouchadam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ouchadam/subscriptions","organizations_url":"https://api.github.com/users/ouchadam/orgs","repos_url":"https://api.github.com/users/ouchadam/repos","events_url":"https://api.github.com/users/ouchadam/events{/privacy}","received_events_url":"https://api.github.com/users/ouchadam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-08T12:36:39Z","updated_at":"2015-07-08T12:38:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/novoda/download-manager/pulls/78","html_url":"https://github.com/novoda/download-manager/pull/78","diff_url":"https://github.com/novoda/download-manager/pull/78.diff","patch_url":"https://github.com/novoda/download-manager/pull/78.patch"},"body":"We found that `isAllowedToDownload` was being called for all queued and running downloads, on **EVERY** download table change. It kind of made sense for running downloads but wasn't really necessary...\r\n\r\nThis PR forces the check to only occur if a download is ready and hasn't been submitted to the downloads exectutor"},"comment":{"url":"https://api.github.com/repos/novoda/download-manager/issues/comments/119561307","html_url":"https://github.com/novoda/download-manager/pull/78#issuecomment-119561307","issue_url":"https://api.github.com/repos/novoda/download-manager/issues/78","id":119561307,"user":{"login":"hal9002","id":2164980,"avatar_url":"https://avatars.githubusercontent.com/u/2164980?v=3","gravatar_id":"","url":"https://api.github.com/users/hal9002","html_url":"https://github.com/hal9002","followers_url":"https://api.github.com/users/hal9002/followers","following_url":"https://api.github.com/users/hal9002/following{/other_user}","gists_url":"https://api.github.com/users/hal9002/gists{/gist_id}","starred_url":"https://api.github.com/users/hal9002/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hal9002/subscriptions","organizations_url":"https://api.github.com/users/hal9002/orgs","repos_url":"https://api.github.com/users/hal9002/repos","events_url":"https://api.github.com/users/hal9002/events{/privacy}","received_events_url":"https://api.github.com/users/hal9002/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T12:38:00Z","updated_at":"2015-07-08T12:38:00Z","body":"\nRefer to this link for build results (access rights to CI server needed): \nhttps://ci.novoda.com/job/download-manager-prb/45/\nTest PASSed."}}