{"action":"created","issue":{"url":"https://api.github.com/repos/thoughtbot/suspenders/issues/532","labels_url":"https://api.github.com/repos/thoughtbot/suspenders/issues/532/labels{/name}","comments_url":"https://api.github.com/repos/thoughtbot/suspenders/issues/532/comments","events_url":"https://api.github.com/repos/thoughtbot/suspenders/issues/532/events","html_url":"https://github.com/thoughtbot/suspenders/pull/532","id":59945522,"number":532,"title":"Remove SMTP assumption","user":{"login":"andyw8","id":13400,"avatar_url":"https://avatars.githubusercontent.com/u/13400?v=3","gravatar_id":"","url":"https://api.github.com/users/andyw8","html_url":"https://github.com/andyw8","followers_url":"https://api.github.com/users/andyw8/followers","following_url":"https://api.github.com/users/andyw8/following{/other_user}","gists_url":"https://api.github.com/users/andyw8/gists{/gist_id}","starred_url":"https://api.github.com/users/andyw8/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyw8/subscriptions","organizations_url":"https://api.github.com/users/andyw8/orgs","repos_url":"https://api.github.com/users/andyw8/repos","events_url":"https://api.github.com/users/andyw8/events{/privacy}","received_events_url":"https://api.github.com/users/andyw8/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-05T12:32:44Z","updated_at":"2015-03-10T00:50:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/thoughtbot/suspenders/pulls/532","html_url":"https://github.com/thoughtbot/suspenders/pull/532","diff_url":"https://github.com/thoughtbot/suspenders/pull/532.diff","patch_url":"https://github.com/thoughtbot/suspenders/pull/532.patch"},"body":"The production config assumes that the SMTP environment variables will be present, but many apps don't need to send mail.\r\n\r\nI'd welcome any suggestions for a more elegant way of handling this. There are four env vars related to SMTP, so `SMTP_ADDRESS` is an arbitrary choice."},"comment":{"url":"https://api.github.com/repos/thoughtbot/suspenders/issues/comments/77975970","html_url":"https://github.com/thoughtbot/suspenders/pull/532#issuecomment-77975970","issue_url":"https://api.github.com/repos/thoughtbot/suspenders/issues/532","id":77975970,"user":{"login":"tute","id":54260,"avatar_url":"https://avatars.githubusercontent.com/u/54260?v=3","gravatar_id":"","url":"https://api.github.com/users/tute","html_url":"https://github.com/tute","followers_url":"https://api.github.com/users/tute/followers","following_url":"https://api.github.com/users/tute/following{/other_user}","gists_url":"https://api.github.com/users/tute/gists{/gist_id}","starred_url":"https://api.github.com/users/tute/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tute/subscriptions","organizations_url":"https://api.github.com/users/tute/orgs","repos_url":"https://api.github.com/users/tute/repos","events_url":"https://api.github.com/users/tute/events{/privacy}","received_events_url":"https://api.github.com/users/tute/received_events","type":"User","site_admin":false},"created_at":"2015-03-10T00:50:05Z","updated_at":"2015-03-10T00:50:05Z","body":"Agreed with @jessieay that this is a perfect fit for an opt-in option, if we had such an architecture.\r\n\r\nI guess the question is to thoughtbot developers: how many times do we need to strip out this config from generated apps vs how many add it in?\r\n\r\nIf it's even, the conditional makes sense to me."}}