{"action":"created","issue":{"url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/19","repository_url":"https://api.github.com/repos/zooniverse/planetary-response-network","labels_url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/19/comments","events_url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/19/events","html_url":"https://github.com/zooniverse/planetary-response-network/pull/19","id":141897473,"number":19,"title":"Refactor","user":{"login":"itsravenous","id":1737376,"avatar_url":"https://avatars.githubusercontent.com/u/1737376?v=3","gravatar_id":"","url":"https://api.github.com/users/itsravenous","html_url":"https://github.com/itsravenous","followers_url":"https://api.github.com/users/itsravenous/followers","following_url":"https://api.github.com/users/itsravenous/following{/other_user}","gists_url":"https://api.github.com/users/itsravenous/gists{/gist_id}","starred_url":"https://api.github.com/users/itsravenous/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/itsravenous/subscriptions","organizations_url":"https://api.github.com/users/itsravenous/orgs","repos_url":"https://api.github.com/users/itsravenous/repos","events_url":"https://api.github.com/users/itsravenous/events{/privacy}","received_events_url":"https://api.github.com/users/itsravenous/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-18T15:25:31Z","updated_at":"2016-03-18T22:08:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/zooniverse/planetary-response-network/pulls/19","html_url":"https://github.com/zooniverse/planetary-response-network/pull/19","diff_url":"https://github.com/zooniverse/planetary-response-network/pull/19.diff","patch_url":"https://github.com/zooniverse/planetary-response-network/pull/19.patch"},"body":"- Creates classes for a lot of the data objects we throw around\r\n- Removes the CSV manifest generation, which as far as I could see was only being used as an intermediate stage before generating JSON subjects. If we do need to create CSV manifests we can add a `toCSV()` method on the Manifest class\r\n- Adds a bunch of tests using [Mocha](https://mochajs.org), [Chai](http://chaijs.com) and [Sinon](http://sinonjs.org/). Still more to add, but we now have a fair amount of coverage\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/comments/198562316","html_url":"https://github.com/zooniverse/planetary-response-network/pull/19#issuecomment-198562316","issue_url":"https://api.github.com/repos/zooniverse/planetary-response-network/issues/19","id":198562316,"user":{"login":"saschaishikawa","id":5863630,"avatar_url":"https://avatars.githubusercontent.com/u/5863630?v=3","gravatar_id":"","url":"https://api.github.com/users/saschaishikawa","html_url":"https://github.com/saschaishikawa","followers_url":"https://api.github.com/users/saschaishikawa/followers","following_url":"https://api.github.com/users/saschaishikawa/following{/other_user}","gists_url":"https://api.github.com/users/saschaishikawa/gists{/gist_id}","starred_url":"https://api.github.com/users/saschaishikawa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saschaishikawa/subscriptions","organizations_url":"https://api.github.com/users/saschaishikawa/orgs","repos_url":"https://api.github.com/users/saschaishikawa/repos","events_url":"https://api.github.com/users/saschaishikawa/events{/privacy}","received_events_url":"https://api.github.com/users/saschaishikawa/received_events","type":"User","site_admin":false},"created_at":"2016-03-18T22:08:10Z","updated_at":"2016-03-18T22:08:10Z","body":"Everything looks good. Replaced the old planet-api-before-after-test script with refactored generate-subjects. Checked that it works both with and without the job queue. @itsravenous, it's ready to merge as far as I'm concerned.\r\n\r\n There's still the issue of how to pass mosaic URIs (currently hard-coded). Should there be a UI within our app that shows which mosaics a user has access to (given their API key)? Still unclear how this would affect Sentinel data. "}}