{"action":"created","issue":{"url":"https://api.github.com/repos/swcarpentry/amy/issues/136","labels_url":"https://api.github.com/repos/swcarpentry/amy/issues/136/labels{/name}","comments_url":"https://api.github.com/repos/swcarpentry/amy/issues/136/comments","events_url":"https://api.github.com/repos/swcarpentry/amy/issues/136/events","html_url":"https://github.com/swcarpentry/amy/pull/136","id":53978187,"number":136,"title":"Merging cohort with event","user":{"login":"gvwilson","id":911566,"avatar_url":"https://avatars.githubusercontent.com/u/911566?v=3","gravatar_id":"","url":"https://api.github.com/users/gvwilson","html_url":"https://github.com/gvwilson","followers_url":"https://api.github.com/users/gvwilson/followers","following_url":"https://api.github.com/users/gvwilson/following{/other_user}","gists_url":"https://api.github.com/users/gvwilson/gists{/gist_id}","starred_url":"https://api.github.com/users/gvwilson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gvwilson/subscriptions","organizations_url":"https://api.github.com/users/gvwilson/orgs","repos_url":"https://api.github.com/users/gvwilson/repos","events_url":"https://api.github.com/users/gvwilson/events{/privacy}","received_events_url":"https://api.github.com/users/gvwilson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-11T01:33:51Z","updated_at":"2015-01-17T23:18:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/swcarpentry/amy/pulls/136","html_url":"https://github.com/swcarpentry/amy/pull/136","diff_url":"https://github.com/swcarpentry/amy/pull/136.diff","patch_url":"https://github.com/swcarpentry/amy/pull/136.patch"},"body":"Addresses #132 (I had time on the plane...)"},"comment":{"url":"https://api.github.com/repos/swcarpentry/amy/issues/comments/70388920","html_url":"https://github.com/swcarpentry/amy/pull/136#issuecomment-70388920","issue_url":"https://api.github.com/repos/swcarpentry/amy/issues/136","id":70388920,"user":{"login":"wking","id":209920,"avatar_url":"https://avatars.githubusercontent.com/u/209920?v=3","gravatar_id":"","url":"https://api.github.com/users/wking","html_url":"https://github.com/wking","followers_url":"https://api.github.com/users/wking/followers","following_url":"https://api.github.com/users/wking/following{/other_user}","gists_url":"https://api.github.com/users/wking/gists{/gist_id}","starred_url":"https://api.github.com/users/wking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wking/subscriptions","organizations_url":"https://api.github.com/users/wking/orgs","repos_url":"https://api.github.com/users/wking/repos","events_url":"https://api.github.com/users/wking/events{/privacy}","received_events_url":"https://api.github.com/users/wking/received_events","type":"User","site_admin":false},"created_at":"2015-01-17T23:18:46Z","updated_at":"2015-01-17T23:18:46Z","body":"The the workshops/ changes are minimal and they look good to me.  I\r\nhaven't checked the migrater.py changes, since I don't have access to\r\nthe input data it's using.\r\n\r\nThere are some things we discussed in #132 that aren't addressed in\r\nthis pull request:\r\n\r\n* It doesn't address the auto-badging and assignment modeling (you'd\r\n  still have to badge individuals by hand).\r\n* It doesn't address the “attendance complete” modeling.\r\n\r\nI feel like those are generic shortcomings of our Event model, and\r\nshouldn't hold back this PR rolling Cohort into Event."}}