{"action":"created","issue":{"url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/1","labels_url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/1/comments","events_url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/1/events","html_url":"https://github.com/maxbschram/railschallenge-city-watch/issues/1","id":69728008,"number":1,"title":"Judge's feedback","user":{"login":"railschallenge","id":11701650,"avatar_url":"https://avatars.githubusercontent.com/u/11701650?v=3","gravatar_id":"","url":"https://api.github.com/users/railschallenge","html_url":"https://github.com/railschallenge","followers_url":"https://api.github.com/users/railschallenge/followers","following_url":"https://api.github.com/users/railschallenge/following{/other_user}","gists_url":"https://api.github.com/users/railschallenge/gists{/gist_id}","starred_url":"https://api.github.com/users/railschallenge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/railschallenge/subscriptions","organizations_url":"https://api.github.com/users/railschallenge/orgs","repos_url":"https://api.github.com/users/railschallenge/repos","events_url":"https://api.github.com/users/railschallenge/events{/privacy}","received_events_url":"https://api.github.com/users/railschallenge/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-21T02:43:05Z","updated_at":"2015-04-23T22:39:58Z","closed_at":null,"body":"Placeholder issue for judges to leave feedback."},"comment":{"url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/comments/95736907","html_url":"https://github.com/maxbschram/railschallenge-city-watch/issues/1#issuecomment-95736907","issue_url":"https://api.github.com/repos/maxbschram/railschallenge-city-watch/issues/1","id":95736907,"user":{"login":"maxbschram","id":11829734,"avatar_url":"https://avatars.githubusercontent.com/u/11829734?v=3","gravatar_id":"","url":"https://api.github.com/users/maxbschram","html_url":"https://github.com/maxbschram","followers_url":"https://api.github.com/users/maxbschram/followers","following_url":"https://api.github.com/users/maxbschram/following{/other_user}","gists_url":"https://api.github.com/users/maxbschram/gists{/gist_id}","starred_url":"https://api.github.com/users/maxbschram/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maxbschram/subscriptions","organizations_url":"https://api.github.com/users/maxbschram/orgs","repos_url":"https://api.github.com/users/maxbschram/repos","events_url":"https://api.github.com/users/maxbschram/events{/privacy}","received_events_url":"https://api.github.com/users/maxbschram/received_events","type":"User","site_admin":false},"created_at":"2015-04-23T22:39:58Z","updated_at":"2015-04-23T22:39:58Z","body":"Thank you so much for the feedback.\r\n\r\nSingle-table inheritance:\r\n* I disabled this so I could have a column named 'type' for the responder type. I could have named it something else, but given that the tests were passing in a hash key of 'type' and the fact that I didn't think STI would be needed it seemed cleaner. Seemed like the lesser of two evils. Do you think keeping STI would have been better practice?\r\n\r\nMigrations:\r\n* I intended to make that column on the responders table be a reference, but by mistake just created data duplication. Thanks for pointing that out.\r\n\r\nI was not familiar with rescue_from. I was looking for something like that, but couldn't find it. Perfect.\r\n\r\nThanks again for your feedback!"}}