{"action":"created","issue":{"url":"https://api.github.com/repos/oppia/oppia/issues/4959","repository_url":"https://api.github.com/repos/oppia/oppia","labels_url":"https://api.github.com/repos/oppia/oppia/issues/4959/labels{/name}","comments_url":"https://api.github.com/repos/oppia/oppia/issues/4959/comments","events_url":"https://api.github.com/repos/oppia/oppia/issues/4959/events","html_url":"https://github.com/oppia/oppia/pull/4959","id":324697096,"number":4959,"title":"Add translator role for the exploration (M1.1 Lesson translation dashboard)","user":{"login":"DubeySandeep","id":16653571,"avatar_url":"https://avatars0.githubusercontent.com/u/16653571?v=4","gravatar_id":"","url":"https://api.github.com/users/DubeySandeep","html_url":"https://github.com/DubeySandeep","followers_url":"https://api.github.com/users/DubeySandeep/followers","following_url":"https://api.github.com/users/DubeySandeep/following{/other_user}","gists_url":"https://api.github.com/users/DubeySandeep/gists{/gist_id}","starred_url":"https://api.github.com/users/DubeySandeep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DubeySandeep/subscriptions","organizations_url":"https://api.github.com/users/DubeySandeep/orgs","repos_url":"https://api.github.com/users/DubeySandeep/repos","events_url":"https://api.github.com/users/DubeySandeep/events{/privacy}","received_events_url":"https://api.github.com/users/DubeySandeep/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"anmolshkl","id":6338799,"avatar_url":"https://avatars0.githubusercontent.com/u/6338799?v=4","gravatar_id":"","url":"https://api.github.com/users/anmolshkl","html_url":"https://github.com/anmolshkl","followers_url":"https://api.github.com/users/anmolshkl/followers","following_url":"https://api.github.com/users/anmolshkl/following{/other_user}","gists_url":"https://api.github.com/users/anmolshkl/gists{/gist_id}","starred_url":"https://api.github.com/users/anmolshkl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anmolshkl/subscriptions","organizations_url":"https://api.github.com/users/anmolshkl/orgs","repos_url":"https://api.github.com/users/anmolshkl/repos","events_url":"https://api.github.com/users/anmolshkl/events{/privacy}","received_events_url":"https://api.github.com/users/anmolshkl/received_events","type":"User","site_admin":false},"assignees":[{"login":"anmolshkl","id":6338799,"avatar_url":"https://avatars0.githubusercontent.com/u/6338799?v=4","gravatar_id":"","url":"https://api.github.com/users/anmolshkl","html_url":"https://github.com/anmolshkl","followers_url":"https://api.github.com/users/anmolshkl/followers","following_url":"https://api.github.com/users/anmolshkl/following{/other_user}","gists_url":"https://api.github.com/users/anmolshkl/gists{/gist_id}","starred_url":"https://api.github.com/users/anmolshkl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anmolshkl/subscriptions","organizations_url":"https://api.github.com/users/anmolshkl/orgs","repos_url":"https://api.github.com/users/anmolshkl/repos","events_url":"https://api.github.com/users/anmolshkl/events{/privacy}","received_events_url":"https://api.github.com/users/anmolshkl/received_events","type":"User","site_admin":false},{"login":"tjiang11","id":11807091,"avatar_url":"https://avatars2.githubusercontent.com/u/11807091?v=4","gravatar_id":"","url":"https://api.github.com/users/tjiang11","html_url":"https://github.com/tjiang11","followers_url":"https://api.github.com/users/tjiang11/followers","following_url":"https://api.github.com/users/tjiang11/following{/other_user}","gists_url":"https://api.github.com/users/tjiang11/gists{/gist_id}","starred_url":"https://api.github.com/users/tjiang11/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tjiang11/subscriptions","organizations_url":"https://api.github.com/users/tjiang11/orgs","repos_url":"https://api.github.com/users/tjiang11/repos","events_url":"https://api.github.com/users/tjiang11/events{/privacy}","received_events_url":"https://api.github.com/users/tjiang11/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-05-20T09:53:39Z","updated_at":"2018-05-25T04:43:59Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/oppia/oppia/pulls/4959","html_url":"https://github.com/oppia/oppia/pull/4959","diff_url":"https://github.com/oppia/oppia/pull/4959.diff","patch_url":"https://github.com/oppia/oppia/pull/4959.patch"},"body":"<!--\r\n  - Thanks for submitting code to Oppia! Please fill out the following as part of\r\n  - your pull request so we can review your code more easily.\r\n  -->\r\n\r\n## Explanation\r\n<!--\r\n  - Explain what your PR does. If this PR fixes an existing bug, please include \r\n  - \"Fixes #bugnum:\" in the explanation so that GitHub can auto-close the issue \r\n  - when this PR is merged.\r\n  -->\r\n- Milestone 1.1:  \r\n  - Adding “translator” role for the exploration.\r\n  - Adding new entity translator_ids in ExplorationSummary and ActivityRights object.\r\n  - Adding `translator_ids` in ExplorationSummaryModel and ExplorationRightsModel.\r\n  - Writing @acl_decorators for can_translate_exploration.\r\n\r\n\r\n## Checklist\r\n- [ ] The PR title starts with \"Fix #bugnum: \", followed by a short, clear summary of the changes.\r\n- [ ] The PR explanation includes the words \"Fixes #bugnum: ...\".\r\n- [x] The linter/Karma presubmit checks have passed.\r\n  - These should run automatically, but if not, you can manually trigger them locally using `python scripts/pre_commit_linter.py` and `bash scripts/run_frontend_tests.sh`.\r\n- [x] The PR is made from a branch that's **not** called \"develop\".\r\n- [x] The PR follows the [style guide](https://github.com/oppia/oppia/wiki/Coding-style-guide).\r\n- [x] The PR is assigned to an appropriate reviewer.\r\n  - If you're a new contributor, please ask on [Gitter](https://gitter.im/oppia/oppia-chat) for someone to assign a reviewer.\r\n  - If you're not sure who the appropriate reviewer is, please assign to the issue's \"owner\" -- see the \"talk-to\" label on the issue.\r\n"},"comment":{"url":"https://api.github.com/repos/oppia/oppia/issues/comments/391939542","html_url":"https://github.com/oppia/oppia/pull/4959#issuecomment-391939542","issue_url":"https://api.github.com/repos/oppia/oppia/issues/4959","id":391939542,"user":{"login":"nithusha21","id":11898234,"avatar_url":"https://avatars2.githubusercontent.com/u/11898234?v=4","gravatar_id":"","url":"https://api.github.com/users/nithusha21","html_url":"https://github.com/nithusha21","followers_url":"https://api.github.com/users/nithusha21/followers","following_url":"https://api.github.com/users/nithusha21/following{/other_user}","gists_url":"https://api.github.com/users/nithusha21/gists{/gist_id}","starred_url":"https://api.github.com/users/nithusha21/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nithusha21/subscriptions","organizations_url":"https://api.github.com/users/nithusha21/orgs","repos_url":"https://api.github.com/users/nithusha21/repos","events_url":"https://api.github.com/users/nithusha21/events{/privacy}","received_events_url":"https://api.github.com/users/nithusha21/received_events","type":"User","site_admin":false},"created_at":"2018-05-25T04:43:59Z","updated_at":"2018-05-25T04:43:59Z","author_association":"MEMBER","body":"The get_json parts LGTM. I'll file an issue regarding 302 responses. I think your point is valid about sending some other response codes for not logged in situations. "}}
{"actor":{"display_login":"nithusha21"}}