{"action":"created","issue":{"url":"https://api.github.com/repos/psu-stewardship/etda/issues/69","repository_url":"https://api.github.com/repos/psu-stewardship/etda","labels_url":"https://api.github.com/repos/psu-stewardship/etda/issues/69/labels{/name}","comments_url":"https://api.github.com/repos/psu-stewardship/etda/issues/69/comments","events_url":"https://api.github.com/repos/psu-stewardship/etda/issues/69/events","html_url":"https://github.com/psu-stewardship/etda/pull/69","id":137236659,"number":69,"title":"Adding in presenters per model.","user":{"login":"cam156","id":1599081,"avatar_url":"https://avatars.githubusercontent.com/u/1599081?v=3","gravatar_id":"","url":"https://api.github.com/users/cam156","html_url":"https://github.com/cam156","followers_url":"https://api.github.com/users/cam156/followers","following_url":"https://api.github.com/users/cam156/following{/other_user}","gists_url":"https://api.github.com/users/cam156/gists{/gist_id}","starred_url":"https://api.github.com/users/cam156/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cam156/subscriptions","organizations_url":"https://api.github.com/users/cam156/orgs","repos_url":"https://api.github.com/users/cam156/repos","events_url":"https://api.github.com/users/cam156/events{/privacy}","received_events_url":"https://api.github.com/users/cam156/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/psu-stewardship/etda/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-02-29T12:06:52Z","updated_at":"2016-02-29T20:38:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/psu-stewardship/etda/pulls/69","html_url":"https://github.com/psu-stewardship/etda/pull/69","diff_url":"https://github.com/psu-stewardship/etda/pull/69.diff","patch_url":"https://github.com/psu-stewardship/etda/pull/69.patch"},"body":"Includes adding in artifacts to refactor the submission presenter\r\n\r\nI'm still unsure what to really call the ThinMint class.  It's a presenter factory Maybe?  Should it just be removed and use the helper for instantiating the AuthorPresenter?\r\n\r\nThis should be discussed before merging... "},"comment":{"url":"https://api.github.com/repos/psu-stewardship/etda/issues/comments/190376904","html_url":"https://github.com/psu-stewardship/etda/pull/69#issuecomment-190376904","issue_url":"https://api.github.com/repos/psu-stewardship/etda/issues/69","id":190376904,"user":{"login":"cam156","id":1599081,"avatar_url":"https://avatars.githubusercontent.com/u/1599081?v=3","gravatar_id":"","url":"https://api.github.com/users/cam156","html_url":"https://github.com/cam156","followers_url":"https://api.github.com/users/cam156/followers","following_url":"https://api.github.com/users/cam156/following{/other_user}","gists_url":"https://api.github.com/users/cam156/gists{/gist_id}","starred_url":"https://api.github.com/users/cam156/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cam156/subscriptions","organizations_url":"https://api.github.com/users/cam156/orgs","repos_url":"https://api.github.com/users/cam156/repos","events_url":"https://api.github.com/users/cam156/events{/privacy}","received_events_url":"https://api.github.com/users/cam156/received_events","type":"User","site_admin":false},"created_at":"2016-02-29T20:38:42Z","updated_at":"2016-02-29T20:38:42Z","body":"@awead thanks for the review.  You raise a good question,  Should all the html be moved into helpers or partials instead of being inside a presenter?  I obviously leaned towards it being in the presenter and the previous iteration had html hard coded in strings in the presenter.  That said I'm ok moving it."}}