{"action":"created","issue":{"url":"https://api.github.com/repos/ndlib/sipity/issues/185","labels_url":"https://api.github.com/repos/ndlib/sipity/issues/185/labels{/name}","comments_url":"https://api.github.com/repos/ndlib/sipity/issues/185/comments","events_url":"https://api.github.com/repos/ndlib/sipity/issues/185/events","html_url":"https://github.com/ndlib/sipity/pull/185","id":59270271,"number":185,"title":"Restoring test coverage","user":{"login":"jeremyf","id":2130,"avatar_url":"https://avatars.githubusercontent.com/u/2130?v=3","gravatar_id":"","url":"https://api.github.com/users/jeremyf","html_url":"https://github.com/jeremyf","followers_url":"https://api.github.com/users/jeremyf/followers","following_url":"https://api.github.com/users/jeremyf/following{/other_user}","gists_url":"https://api.github.com/users/jeremyf/gists{/gist_id}","starred_url":"https://api.github.com/users/jeremyf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jeremyf/subscriptions","organizations_url":"https://api.github.com/users/jeremyf/orgs","repos_url":"https://api.github.com/users/jeremyf/repos","events_url":"https://api.github.com/users/jeremyf/events{/privacy}","received_events_url":"https://api.github.com/users/jeremyf/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-27T17:49:22Z","updated_at":"2015-02-27T17:52:39Z","closed_at":"2015-02-27T17:52:39Z","pull_request":{"url":"https://api.github.com/repos/ndlib/sipity/pulls/185","html_url":"https://github.com/ndlib/sipity/pull/185","diff_url":"https://github.com/ndlib/sipity/pull/185.diff","patch_url":"https://github.com/ndlib/sipity/pull/185.patch"},"body":"## Moving class declaration for clarity\r\n\r\n@68f1a61ea01411b45c57d786ac031b9dcbfc0ba6\r\n\r\nThe incorrect class was at the beginning of the file. This made the\r\ncode more confusing.\r\n\r\n## Improving clarity of intent by renaming\r\n\r\n@2f45288fed87d24a6f09b0b9ac104013d8fb771e\r\n\r\nInstead of the less expressive object, refering to it as an\r\nattachment.\r\n\r\n## Exposing a method to ease presenter logic\r\n\r\n@ed6ac1c151a1dc2e15ac180243f84e06ed4c3e3b\r\n\r\nThis method is to help a simple wrapping element not need to be so\r\nvery nosy about its neighbor.\r\n\r\n## Reducing the presenter's knowledge\r\n\r\n@700713c79645ba577031b71e298e1b2914bdd213\r\n\r\nInstead of digging deep into an attachment, I'm exposing the basic\r\ninformation that I need.\r\n\r\n## Privatizing object to narrow exposure\r\n\r\n@d3b74b14d8769b8c619a0a3f9bcab52fe96fc1f2\r\n\r\nThis object helps in the collaboration of form building and rendering.\r\nIts primary function is to expose a `#delete` attribute along side a\r\nhandful of methods to help render an HTML form.\r\n"},"comment":{"url":"https://api.github.com/repos/ndlib/sipity/issues/comments/76439998","html_url":"https://github.com/ndlib/sipity/pull/185#issuecomment-76439998","issue_url":"https://api.github.com/repos/ndlib/sipity/issues/185","id":76439998,"user":{"login":"jeremyf","id":2130,"avatar_url":"https://avatars.githubusercontent.com/u/2130?v=3","gravatar_id":"","url":"https://api.github.com/users/jeremyf","html_url":"https://github.com/jeremyf","followers_url":"https://api.github.com/users/jeremyf/followers","following_url":"https://api.github.com/users/jeremyf/following{/other_user}","gists_url":"https://api.github.com/users/jeremyf/gists{/gist_id}","starred_url":"https://api.github.com/users/jeremyf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jeremyf/subscriptions","organizations_url":"https://api.github.com/users/jeremyf/orgs","repos_url":"https://api.github.com/users/jeremyf/repos","events_url":"https://api.github.com/users/jeremyf/events{/privacy}","received_events_url":"https://api.github.com/users/jeremyf/received_events","type":"User","site_admin":false},"created_at":"2015-02-27T17:52:39Z","updated_at":"2015-02-27T17:52:39Z","body":"Hold on this."}}