{"action":"created","comment":{"url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/comments/101166557","pull_request_review_id":21883899,"id":101166557,"diff_hunk":"@@ -552,31 +597,36 @@ def _make_field_info(self, field_name, field):\n \n     def get_file_name_from_path(self, field):\n         \"\"\"\n-        Helper for getting filename from string with path to mongoDB storage.\n+        Helper for getting filename from string with path to MongoDB storage.\n+\n         Example of string:\n             asset-v1-RaccoonGang+1+2018+type@asset+block@<filename>\n \n-        Args:\n-            field: The path to file.\n+        Arguments:\n+            field (str): The path to file.\n         Returns:\n             The name of file with an extension.\n         \"\"\"\n         return field.split('@')[-1]\n \n     def get_path_for(self, file_field):\n         \"\"\"\n-        Url retrieved after storing file field in mongoDB look like this:\n+        Return downloaded asset url with slash in start of it.\n+\n+        Url, retrieved after storing of the file field in MongoDB, looks like this:\n             'asset-v1-RaccoonGang+1+2018+type@asset+block@<filename>'\n-        Returns downloaded asset url with slash in start of it\n+\n+        Arguments:\n+            file_field (str): name a file is stored in MongoDB under.\n+        Returns:\n+            Full path of a downloaded asset.\n         \"\"\"\n         if file_field:\n             return os.path.join('/', file_field)\n         return ''\n \n     def get_transcript_download_link(self):\n-        \"\"\"\n-        Returns link for downloading transcript of the current captions language if it exists\n-        \"\"\"\n+        \"\"\"Return link for downloading of a transcript of the current captions' language (if a transcript exists).\"\"\"","path":"video_xblock/video_xblock.py","position":272,"original_position":272,"commit_id":"626c6e1096ec0bd797e43c15bad6251307600389","original_commit_id":"626c6e1096ec0bd797e43c15bad6251307600389","user":{"login":"z4y4ts","id":453047,"avatar_url":"https://avatars.githubusercontent.com/u/453047?v=3","gravatar_id":"","url":"https://api.github.com/users/z4y4ts","html_url":"https://github.com/z4y4ts","followers_url":"https://api.github.com/users/z4y4ts/followers","following_url":"https://api.github.com/users/z4y4ts/following{/other_user}","gists_url":"https://api.github.com/users/z4y4ts/gists{/gist_id}","starred_url":"https://api.github.com/users/z4y4ts/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/z4y4ts/subscriptions","organizations_url":"https://api.github.com/users/z4y4ts/orgs","repos_url":"https://api.github.com/users/z4y4ts/repos","events_url":"https://api.github.com/users/z4y4ts/events{/privacy}","received_events_url":"https://api.github.com/users/z4y4ts/received_events","type":"User","site_admin":false},"body":"Please make it multiline.","created_at":"2017-02-14T23:05:20Z","updated_at":"2017-02-14T23:10:41Z","html_url":"https://github.com/raccoongang/xblock-video/pull/109#discussion_r101166557","pull_request_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109","_links":{"self":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/comments/101166557"},"html":{"href":"https://github.com/raccoongang/xblock-video/pull/109#discussion_r101166557"},"pull_request":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109"}}},"pull_request":{"url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109","id":104989635,"html_url":"https://github.com/raccoongang/xblock-video/pull/109","diff_url":"https://github.com/raccoongang/xblock-video/pull/109.diff","patch_url":"https://github.com/raccoongang/xblock-video/pull/109.patch","issue_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/109","number":109,"state":"open","locked":false,"title":"Fix PEP 257 docstrings violations","user":{"login":"z4y4ts","id":453047,"avatar_url":"https://avatars.githubusercontent.com/u/453047?v=3","gravatar_id":"","url":"https://api.github.com/users/z4y4ts","html_url":"https://github.com/z4y4ts","followers_url":"https://api.github.com/users/z4y4ts/followers","following_url":"https://api.github.com/users/z4y4ts/following{/other_user}","gists_url":"https://api.github.com/users/z4y4ts/gists{/gist_id}","starred_url":"https://api.github.com/users/z4y4ts/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/z4y4ts/subscriptions","organizations_url":"https://api.github.com/users/z4y4ts/orgs","repos_url":"https://api.github.com/users/z4y4ts/repos","events_url":"https://api.github.com/users/z4y4ts/events{/privacy}","received_events_url":"https://api.github.com/users/z4y4ts/received_events","type":"User","site_admin":false},"body":"[Docstrings code style](https://edx.readthedocs.io/projects/edx-developer-guide/en/latest/style_guides/python_guidelines.html#docstrings):\r\n- Follow PEP 257.\r\n- Write docstrings for all modules, classes, and functions.\r\n- Always format docstrings using the multi-line convention, even if there’s only one line of content (see below).\r\n- Use three double-quotes for all docstrings.\r\n- Start with a one-line summary. If you can’t fit a summary in one line, think harder, or refactor the code.\r\n- Write in Sphinx-friendly prose style. Put backquotes around code names (variables, parameters, methods, etc).\r\n- Use imperative mode.","created_at":"2017-02-07T11:52:24Z","updated_at":"2017-02-14T23:10:41Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3655af504362157c6c1f65833513b7d96c318c4e","assignee":{"login":"OPersian","id":10282462,"avatar_url":"https://avatars.githubusercontent.com/u/10282462?v=3","gravatar_id":"","url":"https://api.github.com/users/OPersian","html_url":"https://github.com/OPersian","followers_url":"https://api.github.com/users/OPersian/followers","following_url":"https://api.github.com/users/OPersian/following{/other_user}","gists_url":"https://api.github.com/users/OPersian/gists{/gist_id}","starred_url":"https://api.github.com/users/OPersian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OPersian/subscriptions","organizations_url":"https://api.github.com/users/OPersian/orgs","repos_url":"https://api.github.com/users/OPersian/repos","events_url":"https://api.github.com/users/OPersian/events{/privacy}","received_events_url":"https://api.github.com/users/OPersian/received_events","type":"User","site_admin":false},"assignees":[{"login":"OPersian","id":10282462,"avatar_url":"https://avatars.githubusercontent.com/u/10282462?v=3","gravatar_id":"","url":"https://api.github.com/users/OPersian","html_url":"https://github.com/OPersian","followers_url":"https://api.github.com/users/OPersian/followers","following_url":"https://api.github.com/users/OPersian/following{/other_user}","gists_url":"https://api.github.com/users/OPersian/gists{/gist_id}","starred_url":"https://api.github.com/users/OPersian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OPersian/subscriptions","organizations_url":"https://api.github.com/users/OPersian/orgs","repos_url":"https://api.github.com/users/OPersian/repos","events_url":"https://api.github.com/users/OPersian/events{/privacy}","received_events_url":"https://api.github.com/users/OPersian/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109/commits","review_comments_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109/comments","review_comment_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls/comments{/number}","comments_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/109/comments","statuses_url":"https://api.github.com/repos/raccoongang/xblock-video/statuses/626c6e1096ec0bd797e43c15bad6251307600389","head":{"label":"raccoongang:quality/pydocstyle","ref":"quality/pydocstyle","sha":"626c6e1096ec0bd797e43c15bad6251307600389","user":{"login":"raccoongang","id":11388447,"avatar_url":"https://avatars.githubusercontent.com/u/11388447?v=3","gravatar_id":"","url":"https://api.github.com/users/raccoongang","html_url":"https://github.com/raccoongang","followers_url":"https://api.github.com/users/raccoongang/followers","following_url":"https://api.github.com/users/raccoongang/following{/other_user}","gists_url":"https://api.github.com/users/raccoongang/gists{/gist_id}","starred_url":"https://api.github.com/users/raccoongang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raccoongang/subscriptions","organizations_url":"https://api.github.com/users/raccoongang/orgs","repos_url":"https://api.github.com/users/raccoongang/repos","events_url":"https://api.github.com/users/raccoongang/events{/privacy}","received_events_url":"https://api.github.com/users/raccoongang/received_events","type":"Organization","site_admin":false},"repo":{"id":72450225,"name":"xblock-video","full_name":"raccoongang/xblock-video","owner":{"login":"raccoongang","id":11388447,"avatar_url":"https://avatars.githubusercontent.com/u/11388447?v=3","gravatar_id":"","url":"https://api.github.com/users/raccoongang","html_url":"https://github.com/raccoongang","followers_url":"https://api.github.com/users/raccoongang/followers","following_url":"https://api.github.com/users/raccoongang/following{/other_user}","gists_url":"https://api.github.com/users/raccoongang/gists{/gist_id}","starred_url":"https://api.github.com/users/raccoongang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raccoongang/subscriptions","organizations_url":"https://api.github.com/users/raccoongang/orgs","repos_url":"https://api.github.com/users/raccoongang/repos","events_url":"https://api.github.com/users/raccoongang/events{/privacy}","received_events_url":"https://api.github.com/users/raccoongang/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/raccoongang/xblock-video","description":"Universal Video XBlock with pluggable backend for many video hosting providers","fork":true,"url":"https://api.github.com/repos/raccoongang/xblock-video","forks_url":"https://api.github.com/repos/raccoongang/xblock-video/forks","keys_url":"https://api.github.com/repos/raccoongang/xblock-video/keys{/key_id}","collaborators_url":"https://api.github.com/repos/raccoongang/xblock-video/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/raccoongang/xblock-video/teams","hooks_url":"https://api.github.com/repos/raccoongang/xblock-video/hooks","issue_events_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/events{/number}","events_url":"https://api.github.com/repos/raccoongang/xblock-video/events","assignees_url":"https://api.github.com/repos/raccoongang/xblock-video/assignees{/user}","branches_url":"https://api.github.com/repos/raccoongang/xblock-video/branches{/branch}","tags_url":"https://api.github.com/repos/raccoongang/xblock-video/tags","blobs_url":"https://api.github.com/repos/raccoongang/xblock-video/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/raccoongang/xblock-video/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/raccoongang/xblock-video/git/refs{/sha}","trees_url":"https://api.github.com/repos/raccoongang/xblock-video/git/trees{/sha}","statuses_url":"https://api.github.com/repos/raccoongang/xblock-video/statuses/{sha}","languages_url":"https://api.github.com/repos/raccoongang/xblock-video/languages","stargazers_url":"https://api.github.com/repos/raccoongang/xblock-video/stargazers","contributors_url":"https://api.github.com/repos/raccoongang/xblock-video/contributors","subscribers_url":"https://api.github.com/repos/raccoongang/xblock-video/subscribers","subscription_url":"https://api.github.com/repos/raccoongang/xblock-video/subscription","commits_url":"https://api.github.com/repos/raccoongang/xblock-video/commits{/sha}","git_commits_url":"https://api.github.com/repos/raccoongang/xblock-video/git/commits{/sha}","comments_url":"https://api.github.com/repos/raccoongang/xblock-video/comments{/number}","issue_comment_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/comments{/number}","contents_url":"https://api.github.com/repos/raccoongang/xblock-video/contents/{+path}","compare_url":"https://api.github.com/repos/raccoongang/xblock-video/compare/{base}...{head}","merges_url":"https://api.github.com/repos/raccoongang/xblock-video/merges","archive_url":"https://api.github.com/repos/raccoongang/xblock-video/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/raccoongang/xblock-video/downloads","issues_url":"https://api.github.com/repos/raccoongang/xblock-video/issues{/number}","pulls_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls{/number}","milestones_url":"https://api.github.com/repos/raccoongang/xblock-video/milestones{/number}","notifications_url":"https://api.github.com/repos/raccoongang/xblock-video/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/raccoongang/xblock-video/labels{/name}","releases_url":"https://api.github.com/repos/raccoongang/xblock-video/releases{/id}","deployments_url":"https://api.github.com/repos/raccoongang/xblock-video/deployments","created_at":"2016-10-31T15:30:25Z","updated_at":"2017-02-01T18:02:00Z","pushed_at":"2017-02-14T22:00:11Z","git_url":"git://github.com/raccoongang/xblock-video.git","ssh_url":"git@github.com:raccoongang/xblock-video.git","clone_url":"https://github.com/raccoongang/xblock-video.git","svn_url":"https://github.com/raccoongang/xblock-video","homepage":null,"size":1628,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":3,"forks":3,"open_issues":3,"watchers":0,"default_branch":"dev"}},"base":{"label":"raccoongang:dev","ref":"dev","sha":"36fbadc455242b36bb9fb93264a4b41ec640b07e","user":{"login":"raccoongang","id":11388447,"avatar_url":"https://avatars.githubusercontent.com/u/11388447?v=3","gravatar_id":"","url":"https://api.github.com/users/raccoongang","html_url":"https://github.com/raccoongang","followers_url":"https://api.github.com/users/raccoongang/followers","following_url":"https://api.github.com/users/raccoongang/following{/other_user}","gists_url":"https://api.github.com/users/raccoongang/gists{/gist_id}","starred_url":"https://api.github.com/users/raccoongang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raccoongang/subscriptions","organizations_url":"https://api.github.com/users/raccoongang/orgs","repos_url":"https://api.github.com/users/raccoongang/repos","events_url":"https://api.github.com/users/raccoongang/events{/privacy}","received_events_url":"https://api.github.com/users/raccoongang/received_events","type":"Organization","site_admin":false},"repo":{"id":72450225,"name":"xblock-video","full_name":"raccoongang/xblock-video","owner":{"login":"raccoongang","id":11388447,"avatar_url":"https://avatars.githubusercontent.com/u/11388447?v=3","gravatar_id":"","url":"https://api.github.com/users/raccoongang","html_url":"https://github.com/raccoongang","followers_url":"https://api.github.com/users/raccoongang/followers","following_url":"https://api.github.com/users/raccoongang/following{/other_user}","gists_url":"https://api.github.com/users/raccoongang/gists{/gist_id}","starred_url":"https://api.github.com/users/raccoongang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raccoongang/subscriptions","organizations_url":"https://api.github.com/users/raccoongang/orgs","repos_url":"https://api.github.com/users/raccoongang/repos","events_url":"https://api.github.com/users/raccoongang/events{/privacy}","received_events_url":"https://api.github.com/users/raccoongang/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/raccoongang/xblock-video","description":"Universal Video XBlock with pluggable backend for many video hosting providers","fork":true,"url":"https://api.github.com/repos/raccoongang/xblock-video","forks_url":"https://api.github.com/repos/raccoongang/xblock-video/forks","keys_url":"https://api.github.com/repos/raccoongang/xblock-video/keys{/key_id}","collaborators_url":"https://api.github.com/repos/raccoongang/xblock-video/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/raccoongang/xblock-video/teams","hooks_url":"https://api.github.com/repos/raccoongang/xblock-video/hooks","issue_events_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/events{/number}","events_url":"https://api.github.com/repos/raccoongang/xblock-video/events","assignees_url":"https://api.github.com/repos/raccoongang/xblock-video/assignees{/user}","branches_url":"https://api.github.com/repos/raccoongang/xblock-video/branches{/branch}","tags_url":"https://api.github.com/repos/raccoongang/xblock-video/tags","blobs_url":"https://api.github.com/repos/raccoongang/xblock-video/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/raccoongang/xblock-video/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/raccoongang/xblock-video/git/refs{/sha}","trees_url":"https://api.github.com/repos/raccoongang/xblock-video/git/trees{/sha}","statuses_url":"https://api.github.com/repos/raccoongang/xblock-video/statuses/{sha}","languages_url":"https://api.github.com/repos/raccoongang/xblock-video/languages","stargazers_url":"https://api.github.com/repos/raccoongang/xblock-video/stargazers","contributors_url":"https://api.github.com/repos/raccoongang/xblock-video/contributors","subscribers_url":"https://api.github.com/repos/raccoongang/xblock-video/subscribers","subscription_url":"https://api.github.com/repos/raccoongang/xblock-video/subscription","commits_url":"https://api.github.com/repos/raccoongang/xblock-video/commits{/sha}","git_commits_url":"https://api.github.com/repos/raccoongang/xblock-video/git/commits{/sha}","comments_url":"https://api.github.com/repos/raccoongang/xblock-video/comments{/number}","issue_comment_url":"https://api.github.com/repos/raccoongang/xblock-video/issues/comments{/number}","contents_url":"https://api.github.com/repos/raccoongang/xblock-video/contents/{+path}","compare_url":"https://api.github.com/repos/raccoongang/xblock-video/compare/{base}...{head}","merges_url":"https://api.github.com/repos/raccoongang/xblock-video/merges","archive_url":"https://api.github.com/repos/raccoongang/xblock-video/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/raccoongang/xblock-video/downloads","issues_url":"https://api.github.com/repos/raccoongang/xblock-video/issues{/number}","pulls_url":"https://api.github.com/repos/raccoongang/xblock-video/pulls{/number}","milestones_url":"https://api.github.com/repos/raccoongang/xblock-video/milestones{/number}","notifications_url":"https://api.github.com/repos/raccoongang/xblock-video/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/raccoongang/xblock-video/labels{/name}","releases_url":"https://api.github.com/repos/raccoongang/xblock-video/releases{/id}","deployments_url":"https://api.github.com/repos/raccoongang/xblock-video/deployments","created_at":"2016-10-31T15:30:25Z","updated_at":"2017-02-01T18:02:00Z","pushed_at":"2017-02-14T22:00:11Z","git_url":"git://github.com/raccoongang/xblock-video.git","ssh_url":"git@github.com:raccoongang/xblock-video.git","clone_url":"https://github.com/raccoongang/xblock-video.git","svn_url":"https://github.com/raccoongang/xblock-video","homepage":null,"size":1628,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":3,"forks":3,"open_issues":3,"watchers":0,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109"},"html":{"href":"https://github.com/raccoongang/xblock-video/pull/109"},"issue":{"href":"https://api.github.com/repos/raccoongang/xblock-video/issues/109"},"comments":{"href":"https://api.github.com/repos/raccoongang/xblock-video/issues/109/comments"},"review_comments":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109/comments"},"review_comment":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/raccoongang/xblock-video/pulls/109/commits"},"statuses":{"href":"https://api.github.com/repos/raccoongang/xblock-video/statuses/626c6e1096ec0bd797e43c15bad6251307600389"}}}}
{"actor":{"display_login":"z4y4ts"}}