{"action":"created","issue":{"url":"https://api.github.com/repos/electron/apps/issues/591","repository_url":"https://api.github.com/repos/electron/apps","labels_url":"https://api.github.com/repos/electron/apps/issues/591/labels{/name}","comments_url":"https://api.github.com/repos/electron/apps/issues/591/comments","events_url":"https://api.github.com/repos/electron/apps/issues/591/events","html_url":"https://github.com/electron/apps/pull/591","id":310117361,"number":591,"title":"correct documented minimum icon dimensions","user":{"login":"zeke","id":2289,"avatar_url":"https://avatars1.githubusercontent.com/u/2289?v=4","gravatar_id":"","url":"https://api.github.com/users/zeke","html_url":"https://github.com/zeke","followers_url":"https://api.github.com/users/zeke/followers","following_url":"https://api.github.com/users/zeke/following{/other_user}","gists_url":"https://api.github.com/users/zeke/gists{/gist_id}","starred_url":"https://api.github.com/users/zeke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zeke/subscriptions","organizations_url":"https://api.github.com/users/zeke/orgs","repos_url":"https://api.github.com/users/zeke/repos","events_url":"https://api.github.com/users/zeke/events{/privacy}","received_events_url":"https://api.github.com/users/zeke/received_events","type":"User","site_admin":true},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-03-30T17:05:21Z","updated_at":"2018-04-01T15:36:58Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/electron/apps/pulls/591","html_url":"https://github.com/electron/apps/pull/591","diff_url":"https://github.com/electron/apps/pull/591.diff","patch_url":"https://github.com/electron/apps/pull/591.patch"},"body":"The documented minimum icon size was 256, but it's actually 128.\r\n\r\nUnfortunately there are 67 apps with icons below 256, so we can't easily change that threshold:\r\n\r\nhttps://gist.github.com/zeke/d69001334e91a7122a8d2ca20ab42efa\r\n\r\nTho we could require _newly added apps_ to meet the 256 requirement, and make exceptions for older apps. 🤔 \r\n\r\n@vanessayuenn "},"comment":{"url":"https://api.github.com/repos/electron/apps/issues/comments/377795031","html_url":"https://github.com/electron/apps/pull/591#issuecomment-377795031","issue_url":"https://api.github.com/repos/electron/apps/issues/591","id":377795031,"user":{"login":"ckerr","id":70381,"avatar_url":"https://avatars3.githubusercontent.com/u/70381?v=4","gravatar_id":"","url":"https://api.github.com/users/ckerr","html_url":"https://github.com/ckerr","followers_url":"https://api.github.com/users/ckerr/followers","following_url":"https://api.github.com/users/ckerr/following{/other_user}","gists_url":"https://api.github.com/users/ckerr/gists{/gist_id}","starred_url":"https://api.github.com/users/ckerr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ckerr/subscriptions","organizations_url":"https://api.github.com/users/ckerr/orgs","repos_url":"https://api.github.com/users/ckerr/repos","events_url":"https://api.github.com/users/ckerr/events{/privacy}","received_events_url":"https://api.github.com/users/ckerr/received_events","type":"User","site_admin":true},"created_at":"2018-04-01T15:36:58Z","updated_at":"2018-04-01T15:36:58Z","author_association":"CONTRIBUTOR","body":"+1  on keeping the documented level as-is at 256, and requiring it for new apps"}}
{"actor":{"display_login":"ckerr"}}