{"action":"created","comment":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/comments/78181919","id":78181919,"diff_hunk":"@@ -751,20 +751,28 @@ def _do_extra_actions(api_content, cc_content, request_fields, actions_form, con\n                 raise ValidationError({field: [\"Invalid Key\"]})\n \n \n-def _handle_following_field(form_value, user, cc_content):\n+def _handle_following_field(form_value, user, cc_content, api_content):\n     \"\"\"follow/unfollow thread for the user\"\"\"\n     if form_value:\n         user.follow(cc_content)\n+        if cc_content[\"type\"] == \"thread\":\n+            api_content[\"read\"] = True","path":"lms/djangoapps/discussion_api/api.py","position":22,"original_position":22,"commit_id":"82eb217032c92174759975caf1a3779a2a2660a4","original_commit_id":"82eb217032c92174759975caf1a3779a2a2660a4","user":{"login":"robrap","id":14576445,"avatar_url":"https://avatars.githubusercontent.com/u/14576445?v=3","gravatar_id":"","url":"https://api.github.com/users/robrap","html_url":"https://github.com/robrap","followers_url":"https://api.github.com/users/robrap/followers","following_url":"https://api.github.com/users/robrap/following{/other_user}","gists_url":"https://api.github.com/users/robrap/gists{/gist_id}","starred_url":"https://api.github.com/users/robrap/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/robrap/subscriptions","organizations_url":"https://api.github.com/users/robrap/orgs","repos_url":"https://api.github.com/users/robrap/repos","events_url":"https://api.github.com/users/robrap/events{/privacy}","received_events_url":"https://api.github.com/users/robrap/received_events","type":"User","site_admin":false},"body":"How do you feel about the following comment?\r\n```\r\n# always return read as True as reasonably accurate shortcut, rather than adding additional processing.\r\n```","created_at":"2016-09-09T13:46:00Z","updated_at":"2016-09-09T13:46:01Z","html_url":"https://github.com/edx/edx-platform/pull/13431#discussion_r78181919","pull_request_url":"https://api.github.com/repos/edx/edx-platform/pulls/13431","_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/comments/78181919"},"html":{"href":"https://github.com/edx/edx-platform/pull/13431#discussion_r78181919"},"pull_request":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/13431"}}},"pull_request":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/13431","id":84610501,"html_url":"https://github.com/edx/edx-platform/pull/13431","diff_url":"https://github.com/edx/edx-platform/pull/13431.diff","patch_url":"https://github.com/edx/edx-platform/pull/13431.patch","issue_url":"https://api.github.com/repos/edx/edx-platform/issues/13431","number":13431,"state":"open","locked":false,"title":"MA-2663: Donot mark thread unread on any action but creation","user":{"login":"wajeeha-khalid","id":6745069,"avatar_url":"https://avatars.githubusercontent.com/u/6745069?v=3","gravatar_id":"","url":"https://api.github.com/users/wajeeha-khalid","html_url":"https://github.com/wajeeha-khalid","followers_url":"https://api.github.com/users/wajeeha-khalid/followers","following_url":"https://api.github.com/users/wajeeha-khalid/following{/other_user}","gists_url":"https://api.github.com/users/wajeeha-khalid/gists{/gist_id}","starred_url":"https://api.github.com/users/wajeeha-khalid/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wajeeha-khalid/subscriptions","organizations_url":"https://api.github.com/users/wajeeha-khalid/orgs","repos_url":"https://api.github.com/users/wajeeha-khalid/repos","events_url":"https://api.github.com/users/wajeeha-khalid/events{/privacy}","received_events_url":"https://api.github.com/users/wajeeha-khalid/received_events","type":"User","site_admin":false},"body":"## [MA-2663](https://openedx.atlassian.net/browse/MA-2663)\r\n\r\nDo not mark thread back to unread on any of the following action:\r\n- follow\r\n- endorse\r\n- mark as answer\r\n- vote\r\n- edit \r\n- delete\r\n- report\r\n\r\n### Testing\r\n- [x] Unit\r\n\r\n### Reviewers\r\nIf you've been tagged for review, please check your corresponding box once you've given the :+1:.\r\n- [ ] Code review: @jcdyer \r\n- [ ] Code review: @robrap \r\n\r\n### Post-review\r\n- [ ] Squash commits","created_at":"2016-09-09T08:14:40Z","updated_at":"2016-09-09T13:46:01Z","closed_at":null,"merged_at":null,"merge_commit_sha":"0deeace46083770dd82a285b832efbce7acfbfee","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/edx/edx-platform/pulls/13431/commits","review_comments_url":"https://api.github.com/repos/edx/edx-platform/pulls/13431/comments","review_comment_url":"https://api.github.com/repos/edx/edx-platform/pulls/comments{/number}","comments_url":"https://api.github.com/repos/edx/edx-platform/issues/13431/comments","statuses_url":"https://api.github.com/repos/edx/edx-platform/statuses/82eb217032c92174759975caf1a3779a2a2660a4","head":{"label":"edx:jia/MA-2663","ref":"jia/MA-2663","sha":"82eb217032c92174759975caf1a3779a2a2660a4","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?v=3","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":10391073,"name":"edx-platform","full_name":"edx/edx-platform","owner":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?v=3","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/edx/edx-platform","description":"The Open edX platform, the software that powers edX!","fork":false,"url":"https://api.github.com/repos/edx/edx-platform","forks_url":"https://api.github.com/repos/edx/edx-platform/forks","keys_url":"https://api.github.com/repos/edx/edx-platform/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/edx-platform/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/edx-platform/teams","hooks_url":"https://api.github.com/repos/edx/edx-platform/hooks","issue_events_url":"https://api.github.com/repos/edx/edx-platform/issues/events{/number}","events_url":"https://api.github.com/repos/edx/edx-platform/events","assignees_url":"https://api.github.com/repos/edx/edx-platform/assignees{/user}","branches_url":"https://api.github.com/repos/edx/edx-platform/branches{/branch}","tags_url":"https://api.github.com/repos/edx/edx-platform/tags","blobs_url":"https://api.github.com/repos/edx/edx-platform/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/edx-platform/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/edx-platform/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/edx-platform/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/edx-platform/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/edx-platform/languages","stargazers_url":"https://api.github.com/repos/edx/edx-platform/stargazers","contributors_url":"https://api.github.com/repos/edx/edx-platform/contributors","subscribers_url":"https://api.github.com/repos/edx/edx-platform/subscribers","subscription_url":"https://api.github.com/repos/edx/edx-platform/subscription","commits_url":"https://api.github.com/repos/edx/edx-platform/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/edx-platform/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/edx-platform/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/edx-platform/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/edx-platform/contents/{+path}","compare_url":"https://api.github.com/repos/edx/edx-platform/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/edx-platform/merges","archive_url":"https://api.github.com/repos/edx/edx-platform/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/edx-platform/downloads","issues_url":"https://api.github.com/repos/edx/edx-platform/issues{/number}","pulls_url":"https://api.github.com/repos/edx/edx-platform/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/edx-platform/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/edx-platform/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/edx-platform/labels{/name}","releases_url":"https://api.github.com/repos/edx/edx-platform/releases{/id}","deployments_url":"https://api.github.com/repos/edx/edx-platform/deployments","created_at":"2013-05-30T20:20:38Z","updated_at":"2016-09-09T06:50:50Z","pushed_at":"2016-09-09T13:32:25Z","git_url":"git://github.com/edx/edx-platform.git","ssh_url":"git@github.com:edx/edx-platform.git","clone_url":"https://github.com/edx/edx-platform.git","svn_url":"https://github.com/edx/edx-platform","homepage":"http://open.edx.org/","size":822744,"stargazers_count":2757,"watchers_count":2757,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":1656,"mirror_url":null,"open_issues_count":136,"forks":1656,"open_issues":136,"watchers":2757,"default_branch":"master"}},"base":{"label":"edx:master","ref":"master","sha":"6aaf442d2fc2da3cfb69d727b89343838163aca6","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?v=3","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":10391073,"name":"edx-platform","full_name":"edx/edx-platform","owner":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?v=3","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/edx/edx-platform","description":"The Open edX platform, the software that powers edX!","fork":false,"url":"https://api.github.com/repos/edx/edx-platform","forks_url":"https://api.github.com/repos/edx/edx-platform/forks","keys_url":"https://api.github.com/repos/edx/edx-platform/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/edx-platform/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/edx-platform/teams","hooks_url":"https://api.github.com/repos/edx/edx-platform/hooks","issue_events_url":"https://api.github.com/repos/edx/edx-platform/issues/events{/number}","events_url":"https://api.github.com/repos/edx/edx-platform/events","assignees_url":"https://api.github.com/repos/edx/edx-platform/assignees{/user}","branches_url":"https://api.github.com/repos/edx/edx-platform/branches{/branch}","tags_url":"https://api.github.com/repos/edx/edx-platform/tags","blobs_url":"https://api.github.com/repos/edx/edx-platform/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/edx-platform/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/edx-platform/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/edx-platform/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/edx-platform/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/edx-platform/languages","stargazers_url":"https://api.github.com/repos/edx/edx-platform/stargazers","contributors_url":"https://api.github.com/repos/edx/edx-platform/contributors","subscribers_url":"https://api.github.com/repos/edx/edx-platform/subscribers","subscription_url":"https://api.github.com/repos/edx/edx-platform/subscription","commits_url":"https://api.github.com/repos/edx/edx-platform/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/edx-platform/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/edx-platform/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/edx-platform/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/edx-platform/contents/{+path}","compare_url":"https://api.github.com/repos/edx/edx-platform/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/edx-platform/merges","archive_url":"https://api.github.com/repos/edx/edx-platform/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/edx-platform/downloads","issues_url":"https://api.github.com/repos/edx/edx-platform/issues{/number}","pulls_url":"https://api.github.com/repos/edx/edx-platform/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/edx-platform/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/edx-platform/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/edx-platform/labels{/name}","releases_url":"https://api.github.com/repos/edx/edx-platform/releases{/id}","deployments_url":"https://api.github.com/repos/edx/edx-platform/deployments","created_at":"2013-05-30T20:20:38Z","updated_at":"2016-09-09T06:50:50Z","pushed_at":"2016-09-09T13:32:25Z","git_url":"git://github.com/edx/edx-platform.git","ssh_url":"git@github.com:edx/edx-platform.git","clone_url":"https://github.com/edx/edx-platform.git","svn_url":"https://github.com/edx/edx-platform","homepage":"http://open.edx.org/","size":822744,"stargazers_count":2757,"watchers_count":2757,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":1656,"mirror_url":null,"open_issues_count":136,"forks":1656,"open_issues":136,"watchers":2757,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/13431"},"html":{"href":"https://github.com/edx/edx-platform/pull/13431"},"issue":{"href":"https://api.github.com/repos/edx/edx-platform/issues/13431"},"comments":{"href":"https://api.github.com/repos/edx/edx-platform/issues/13431/comments"},"review_comments":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/13431/comments"},"review_comment":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/13431/commits"},"statuses":{"href":"https://api.github.com/repos/edx/edx-platform/statuses/82eb217032c92174759975caf1a3779a2a2660a4"}}}}
{"actor":{"display_login":"robrap"}}