{"action":"created","issue":{"url":"https://api.github.com/repos/labrad/scalabrad-web/issues/129","labels_url":"https://api.github.com/repos/labrad/scalabrad-web/issues/129/labels{/name}","comments_url":"https://api.github.com/repos/labrad/scalabrad-web/issues/129/comments","events_url":"https://api.github.com/repos/labrad/scalabrad-web/issues/129/events","html_url":"https://github.com/labrad/scalabrad-web/pull/129","id":124878420,"number":129,"title":"Move activities and places into their own modules.","user":{"login":"maffoo","id":129268,"avatar_url":"https://avatars.githubusercontent.com/u/129268?v=3","gravatar_id":"","url":"https://api.github.com/users/maffoo","html_url":"https://github.com/maffoo","followers_url":"https://api.github.com/users/maffoo/followers","following_url":"https://api.github.com/users/maffoo/following{/other_user}","gists_url":"https://api.github.com/users/maffoo/gists{/gist_id}","starred_url":"https://api.github.com/users/maffoo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maffoo/subscriptions","organizations_url":"https://api.github.com/users/maffoo/orgs","repos_url":"https://api.github.com/users/maffoo/repos","events_url":"https://api.github.com/users/maffoo/events{/privacy}","received_events_url":"https://api.github.com/users/maffoo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-01-05T01:53:13Z","updated_at":"2016-01-25T17:40:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/labrad/scalabrad-web/pulls/129","html_url":"https://github.com/labrad/scalabrad-web/pull/129","diff_url":"https://github.com/labrad/scalabrad-web/pull/129.diff","patch_url":"https://github.com/labrad/scalabrad-web/pull/129.patch"},"body":"Pull the Activity classes out into a separate module instead of putting them inside an event callback in the main app module. Among other things, this forces us to declare all the dependencies for each activity in its constructor, rather than close over variables from the outer scope when creating activities. We also put the functions for generating urls into a single location so they can be more self-contained."},"comment":{"url":"https://api.github.com/repos/labrad/scalabrad-web/issues/comments/174598234","html_url":"https://github.com/labrad/scalabrad-web/pull/129#issuecomment-174598234","issue_url":"https://api.github.com/repos/labrad/scalabrad-web/issues/129","id":174598234,"user":{"login":"joshmutus","id":8173925,"avatar_url":"https://avatars.githubusercontent.com/u/8173925?v=3","gravatar_id":"","url":"https://api.github.com/users/joshmutus","html_url":"https://github.com/joshmutus","followers_url":"https://api.github.com/users/joshmutus/followers","following_url":"https://api.github.com/users/joshmutus/following{/other_user}","gists_url":"https://api.github.com/users/joshmutus/gists{/gist_id}","starred_url":"https://api.github.com/users/joshmutus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshmutus/subscriptions","organizations_url":"https://api.github.com/users/joshmutus/orgs","repos_url":"https://api.github.com/users/joshmutus/repos","events_url":"https://api.github.com/users/joshmutus/events{/privacy}","received_events_url":"https://api.github.com/users/joshmutus/received_events","type":"User","site_admin":false},"created_at":"2016-01-25T17:40:01Z","updated_at":"2016-01-25T17:40:01Z","body":"LGTM"}}