GitHub PullRequestReviewCommentEvent at 2014-06-12 17:57:28
Repository: edx-platform
Actor: sarina
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/comments/13716006","id":13716006,"diff_hunk":"@@ -62,15 +64,15 @@ def handle(self, *args, **options):\n             student = User.objects.get(username=user, courseenrollment__course_id=course_id)\n \n         print \"Fetching course data for {0}\".format(course_id)\n-        course = modulestore().get_course(course_id, depth=2)\n+        course = CourseKey.from_string(options['course'])\n \n         if not options['noop']:\n             # Add the certificate request to the queue\n             xq = XQueueCertInterface()\n             if options['insecure']:\n                 xq.use_https = False\n-            ret = xq.regen_cert(student, course_id, course=course, \n-                                forced_grade=options['grade_value'], \n+            ret = xq.regen_cert(student, course_id, course=course,","path":"lms/djangoapps/certificates/management/commands/regenerate_user.py","position":23,"original_position":23,"commit_id":"04634afb76f92741e1e1499cf8d689e82d65df63","original_commit_id":"04634afb76f92741e1e1499cf8d689e82d65df63","user":{"login":"sarina","id":1985317,"avatar_url":"https://avatars.githubusercontent.com/u/1985317?","gravatar_id":"136faf64ffa4486f0038265b06756f52","url":"https://api.github.com/users/sarina","html_url":"https://github.com/sarina","followers_url":"https://api.github.com/users/sarina/followers","following_url":"https://api.github.com/users/sarina/following{/other_user}","gists_url":"https://api.github.com/users/sarina/gists{/gist_id}","starred_url":"https://api.github.com/users/sarina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sarina/subscriptions","organizations_url":"https://api.github.com/users/sarina/orgs","repos_url":"https://api.github.com/users/sarina/repos","events_url":"https://api.github.com/users/sarina/events{/privacy}","received_events_url":"https://api.github.com/users/sarina/received_events","type":"User","site_admin":false},"body":"Are you sure this is right? `course` was previously an instance of a course, but is now a `CourseKey`\r\n\r\nYou might want to take a look at what we did in other management commands to get the CourseKey, eg https://github.com/edx/edx-platform/blob/master/lms/djangoapps/certificates/management/commands/gen_cert_report.py#L50","created_at":"2014-06-12T17:57:28Z","updated_at":"2014-06-12T17:57:28Z","html_url":"https://github.com/edx/edx-platform/pull/4075#discussion_r13716006","pull_request_url":"https://api.github.com/repos/edx/edx-platform/pulls/4075","_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/comments/13716006"},"html":{"href":"https://github.com/edx/edx-platform/pull/4075#discussion_r13716006"},"pull_request":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/4075"}}},"pull_request":{"url":"https://api.github.com/repos/edx/edx-platform/pulls/4075","id":17068212,"html_url":"https://github.com/edx/edx-platform/pull/4075","diff_url":"https://github.com/edx/edx-platform/pull/4075.diff","patch_url":"https://github.com/edx/edx-platform/pull/4075.patch","issue_url":"https://api.github.com/repos/edx/edx-platform/issues/4075","number":4075,"state":"open","title":"Transition to Oqaque Keys","user":{"login":"hkim823","id":1074229,"avatar_url":"https://avatars.githubusercontent.com/u/1074229?","gravatar_id":"26c9dbc89ee066c7f11e51e26c93a72d","url":"https://api.github.com/users/hkim823","html_url":"https://github.com/hkim823","followers_url":"https://api.github.com/users/hkim823/followers","following_url":"https://api.github.com/users/hkim823/following{/other_user}","gists_url":"https://api.github.com/users/hkim823/gists{/gist_id}","starred_url":"https://api.github.com/users/hkim823/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hkim823/subscriptions","organizations_url":"https://api.github.com/users/hkim823/orgs","repos_url":"https://api.github.com/users/hkim823/repos","events_url":"https://api.github.com/users/hkim823/events{/privacy}","received_events_url":"https://api.github.com/users/hkim823/received_events","type":"User","site_admin":false},"body":null,"created_at":"2014-06-12T17:26:43Z","updated_at":"2014-06-12T17:57:28Z","closed_at":null,"merged_at":null,"merge_commit_sha":"c2940d741537d2f4bbbb56c2b6cbef88603e7fbb","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/edx/edx-platform/pulls/4075/commits","review_comments_url":"https://api.github.com/repos/edx/edx-platform/pulls/4075/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/4075/comments","statuses_url":"https://api.github.com/repos/edx/edx-platform/statuses/04634afb76f92741e1e1499cf8d689e82d65df63","head":{"label":"edx:han/regenerate_user","ref":"han/regenerate_user","sha":"04634afb76f92741e1e1499cf8d689e82d65df63","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?","gravatar_id":"e1f6c6dc8772c47c0261cd5df2ed2814","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?","gravatar_id":"e1f6c6dc8772c47c0261cd5df2ed2814","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":"Open edX, the edX platform that powers http://www.edx.org","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}","created_at":"2013-05-30T20:20:38Z","updated_at":"2014-06-12T17:52:28Z","pushed_at":"2014-06-12T17:52:28Z","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://code.edx.org/","size":1150789,"stargazers_count":1250,"watchers_count":1250,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":650,"mirror_url":null,"open_issues_count":118,"forks":650,"open_issues":118,"watchers":1250,"default_branch":"master"}},"base":{"label":"edx:master","ref":"master","sha":"acf298e9ade71fb9d42bf06f39786bdaa8351e99","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars.githubusercontent.com/u/3179841?","gravatar_id":"e1f6c6dc8772c47c0261cd5df2ed2814","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?","gravatar_id":"e1f6c6dc8772c47c0261cd5df2ed2814","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":"Open edX, the edX platform that powers http://www.edx.org","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}","created_at":"2013-05-30T20:20:38Z","updated_at":"2014-06-12T17:52:28Z","pushed_at":"2014-06-12T17:52:28Z","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://code.edx.org/","size":1150789,"stargazers_count":1250,"watchers_count":1250,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":650,"mirror_url":null,"open_issues_count":118,"forks":650,"open_issues":118,"watchers":1250,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/4075"},"html":{"href":"https://github.com/edx/edx-platform/pull/4075"},"issue":{"href":"https://api.github.com/repos/edx/edx-platform/issues/4075"},"comments":{"href":"https://api.github.com/repos/edx/edx-platform/issues/4075/comments"},"review_comments":{"href":"https://api.github.com/repos/edx/edx-platform/pulls/4075/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/4075/commits"},"statuses":{"href":"https://api.github.com/repos/edx/edx-platform/statuses/04634afb76f92741e1e1499cf8d689e82d65df63"}}}}