{"action":"created","issue":{"url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/93","labels_url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/93/labels{/name}","comments_url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/93/comments","events_url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/93/events","html_url":"https://github.com/mikaelbr/node-notifier/issues/93","id":128674041,"number":93,"title":"Use image url for notifier icon parameter","user":{"login":"kingdido999","id":3196372,"avatar_url":"https://avatars.githubusercontent.com/u/3196372?v=3","gravatar_id":"","url":"https://api.github.com/users/kingdido999","html_url":"https://github.com/kingdido999","followers_url":"https://api.github.com/users/kingdido999/followers","following_url":"https://api.github.com/users/kingdido999/following{/other_user}","gists_url":"https://api.github.com/users/kingdido999/gists{/gist_id}","starred_url":"https://api.github.com/users/kingdido999/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kingdido999/subscriptions","organizations_url":"https://api.github.com/users/kingdido999/orgs","repos_url":"https://api.github.com/users/kingdido999/repos","events_url":"https://api.github.com/users/kingdido999/events{/privacy}","received_events_url":"https://api.github.com/users/kingdido999/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-25T23:34:21Z","updated_at":"2016-01-26T05:23:25Z","closed_at":null,"body":"This is just a feature request. For the notify method, I want to display an icon from url. I know I could download the image from url, pass its path to `icon` parameter and then delete the file... I'm wondering if it would be possible to pass in an url as icon parameter?"},"comment":{"url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/comments/174838291","html_url":"https://github.com/mikaelbr/node-notifier/issues/93#issuecomment-174838291","issue_url":"https://api.github.com/repos/mikaelbr/node-notifier/issues/93","id":174838291,"user":{"login":"kurisubrooks","id":6291467,"avatar_url":"https://avatars.githubusercontent.com/u/6291467?v=3","gravatar_id":"","url":"https://api.github.com/users/kurisubrooks","html_url":"https://github.com/kurisubrooks","followers_url":"https://api.github.com/users/kurisubrooks/followers","following_url":"https://api.github.com/users/kurisubrooks/following{/other_user}","gists_url":"https://api.github.com/users/kurisubrooks/gists{/gist_id}","starred_url":"https://api.github.com/users/kurisubrooks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kurisubrooks/subscriptions","organizations_url":"https://api.github.com/users/kurisubrooks/orgs","repos_url":"https://api.github.com/users/kurisubrooks/repos","events_url":"https://api.github.com/users/kurisubrooks/events{/privacy}","received_events_url":"https://api.github.com/users/kurisubrooks/received_events","type":"User","site_admin":false},"created_at":"2016-01-26T05:23:25Z","updated_at":"2016-01-26T05:23:25Z","body":"I'm worried that this will just add some more problems to node-notifier, and make the notification longer to load. The way we'd accomplish this is by checking if the icon is a url, then we'd download it to a temp folder, then make it use this temporary icon, deliver the notification, delete it after so it doesn't waste space on disk.\r\n\r\nAdd time to the loading of the notification, send a network request, deal with automated file permissions...\r\nI don't think this is a very good idea. It's easier just to download a single icon, and include it in code."}}