{"action":"created","issue":{"url":"https://api.github.com/repos/magfest/magprime/issues/50","labels_url":"https://api.github.com/repos/magfest/magprime/issues/50/labels{/name}","comments_url":"https://api.github.com/repos/magfest/magprime/issues/50/comments","events_url":"https://api.github.com/repos/magfest/magprime/issues/50/events","html_url":"https://github.com/magfest/magprime/pull/50","id":123560889,"number":50,"title":"add 2016 swag email","user":{"login":"binary1230","id":5413064,"avatar_url":"https://avatars.githubusercontent.com/u/5413064?v=3","gravatar_id":"","url":"https://api.github.com/users/binary1230","html_url":"https://github.com/binary1230","followers_url":"https://api.github.com/users/binary1230/followers","following_url":"https://api.github.com/users/binary1230/following{/other_user}","gists_url":"https://api.github.com/users/binary1230/gists{/gist_id}","starred_url":"https://api.github.com/users/binary1230/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/binary1230/subscriptions","organizations_url":"https://api.github.com/users/binary1230/orgs","repos_url":"https://api.github.com/users/binary1230/repos","events_url":"https://api.github.com/users/binary1230/events{/privacy}","received_events_url":"https://api.github.com/users/binary1230/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-22T21:39:48Z","updated_at":"2015-12-23T22:14:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/magfest/magprime/pulls/50","html_url":"https://github.com/magfest/magprime/pull/50","diff_url":"https://github.com/magfest/magprime/pull/50.diff","patch_url":"https://github.com/magfest/magprime/pull/50.patch"},"body":"now OK for merging pending review\r\n\r\n@nickthenewbie1 wants to get this out asap, just need to bolt down the last bit and test out the HTML.\r\n\r\nThe HTML contains some hard coded dates in it which we'd need to update next year."},"comment":{"url":"https://api.github.com/repos/magfest/magprime/issues/comments/167000714","html_url":"https://github.com/magfest/magprime/pull/50#issuecomment-167000714","issue_url":"https://api.github.com/repos/magfest/magprime/issues/50","id":167000714,"user":{"login":"nickthenewbie1","id":7833809,"avatar_url":"https://avatars.githubusercontent.com/u/7833809?v=3","gravatar_id":"","url":"https://api.github.com/users/nickthenewbie1","html_url":"https://github.com/nickthenewbie1","followers_url":"https://api.github.com/users/nickthenewbie1/followers","following_url":"https://api.github.com/users/nickthenewbie1/following{/other_user}","gists_url":"https://api.github.com/users/nickthenewbie1/gists{/gist_id}","starred_url":"https://api.github.com/users/nickthenewbie1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nickthenewbie1/subscriptions","organizations_url":"https://api.github.com/users/nickthenewbie1/orgs","repos_url":"https://api.github.com/users/nickthenewbie1/repos","events_url":"https://api.github.com/users/nickthenewbie1/events{/privacy}","received_events_url":"https://api.github.com/users/nickthenewbie1/received_events","type":"User","site_admin":false},"created_at":"2015-12-23T22:14:23Z","updated_at":"2015-12-23T22:14:23Z","body":"We did a mailinator test, and it rendered not so perfectly in mailinator,\nbut forwarded to gmail it looked just fine. This may be impetus to look\ninto more madmimi integration next year if we want to more regularly send\npretty e-mails.\n\nI understand not wanting to annoy the supporters, though this e-mail is\ndouble duty of advertising the swag, and giving people who already ordered\nit a preview of what to expect. You're right that we can revisit the\nconditionals next year.\n\n--\nNicholas Marinelli\nChief Operating Officer\nMAGFest\nhttp://magfest.org\n\nOn Wed, Dec 23, 2015 at 5:06 PM, Eli Courtwright <notifications@github.com>\nwrote:\n\n> This looks good, though I still think we should add in a.amount_extra <\n> c.SUPPORTER_LEVEL or at least a.amount_extra < c.SEASON_LEVEL or else we\n> risk annoying people who already gave us money by advertising things they\n> already bought. Not a dealbreaker though, so feel free to merge this\n> without that addition if you disagree.\n>\n> A backlog issue in this repo for adding conditionals to the email body\n> would be nice, so we can take the time to do a more customized job next\n> year, but that's also not something that needs to hold this up.\n>\n> My final suggestion is to test this out on mailinator from the staging\n> server after it gets merged, just to make sure that the email renders in a\n> reasonable way before it gets blasted out to thousand of people. You can\n> also test from GMail if you set the value of c.DEVELOPER_EMAIL in\n> development.ini, because that user is ALSO able to receive emails from\n> the staging server for just this sort of reason.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/magfest/magprime/pull/50#issuecomment-166999647>.\n>\n"}}