{"action":"created","issue":{"url":"https://api.github.com/repos/edx/edx-platform/issues/5460","labels_url":"https://api.github.com/repos/edx/edx-platform/issues/5460/labels{/name}","comments_url":"https://api.github.com/repos/edx/edx-platform/issues/5460/comments","events_url":"https://api.github.com/repos/edx/edx-platform/issues/5460/events","html_url":"https://github.com/edx/edx-platform/pull/5460","id":44583652,"number":5460,"title":"Do event tracking for major forum events.","user":{"login":"Kelketek","id":1099483,"avatar_url":"https://avatars.githubusercontent.com/u/1099483?v=3","gravatar_id":"","url":"https://api.github.com/users/Kelketek","html_url":"https://github.com/Kelketek","followers_url":"https://api.github.com/users/Kelketek/followers","following_url":"https://api.github.com/users/Kelketek/following{/other_user}","gists_url":"https://api.github.com/users/Kelketek/gists{/gist_id}","starred_url":"https://api.github.com/users/Kelketek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kelketek/subscriptions","organizations_url":"https://api.github.com/users/Kelketek/orgs","repos_url":"https://api.github.com/users/Kelketek/repos","events_url":"https://api.github.com/users/Kelketek/events{/privacy}","received_events_url":"https://api.github.com/users/Kelketek/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/edx/edx-platform/labels/engineering%20review","name":"engineering review","color":"c7def8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":67,"created_at":"2014-10-01T16:17:34Z","updated_at":"2015-02-25T19:27:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/5460","html_url":"https://github.com/edx/edx-platform/pull/5460","diff_url":"https://github.com/edx/edx-platform/pull/5460.diff","patch_url":"https://github.com/edx/edx-platform/pull/5460.patch"},"body":"This PR is intended to supersede #4840 . Please close that one and continue the discussion here.\r\n\r\nThis PR is a WIP because the specification provided at https://openedx.atlassian.net/wiki/pages/viewpage.action?spaceKey=AN&title=Forum+Events has left me with a few questions, and I'm not sure that this implementation conforms entirely to the vision of what was stated. Most of it has been implemented.\r\n\r\nThe tests are also currently failing because tests which test the comment functionality did not anticipate the thread ID needing to be accessed, but these events now require it, so those tests will need patching to permit this. I also added a few new tests which will need similar fixing. But I'd rather start getting feedback now.\r\n\r\nA few issues:\r\n\r\n1. How would one get the category id and name? I'm not really sure what these are to begin with, as I don't see them as fields on the comments.\r\n2. How would one get the origin? It doesn't seem like there's a sane way to do that at the moment. Would there need to be modification to the existing views to post the enum value we want? Would there be any values other than 'courseware' and 'forum'?\r\n3. Some of the keys in the spec appear oddly inconsistent, though I may be missing something. For example: 'discussion.id' vs 'category_id'.\r\n\r\nFeedback would be appreciated. Thank you.\r\n\r\n* Partner information: 3rd party-hosted open edX instance, for an edX solutions client (so will need to get some sort of solution merged in)\r\n* Merge deadline: Oct 22nd (soft requirement to minimize code drift, we maintain it on the client fork in the meantime)\r\n\r\nPing @sarina "},"comment":{"url":"https://api.github.com/repos/edx/edx-platform/issues/comments/76036288","html_url":"https://github.com/edx/edx-platform/pull/5460#issuecomment-76036288","issue_url":"https://api.github.com/repos/edx/edx-platform/issues/5460","id":76036288,"user":{"login":"Kelketek","id":1099483,"avatar_url":"https://avatars.githubusercontent.com/u/1099483?v=3","gravatar_id":"","url":"https://api.github.com/users/Kelketek","html_url":"https://github.com/Kelketek","followers_url":"https://api.github.com/users/Kelketek/followers","following_url":"https://api.github.com/users/Kelketek/following{/other_user}","gists_url":"https://api.github.com/users/Kelketek/gists{/gist_id}","starred_url":"https://api.github.com/users/Kelketek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kelketek/subscriptions","organizations_url":"https://api.github.com/users/Kelketek/orgs","repos_url":"https://api.github.com/users/Kelketek/repos","events_url":"https://api.github.com/users/Kelketek/events{/privacy}","received_events_url":"https://api.github.com/users/Kelketek/received_events","type":"User","site_admin":false},"created_at":"2015-02-25T19:27:23Z","updated_at":"2015-02-25T19:27:23Z","body":"Agh! You're killin' me. I'll get this up later today, then."}}