{"action":"created","comment":{"url":"https://api.github.com/repos/fossasia/yaydoc/pulls/comments/126282874","pull_request_review_id":48757267,"id":126282874,"diff_hunk":"@@ -29,49 +33,52 @@ INVENV=$(python -c 'import sys; print (\"true\" if hasattr(sys, \"real_prefix\") els\n # and change current directory to git repository.\n if [ \"${WEBUI:-false}\" == \"true\" ]; then\n   BASE=$(pwd)\n+  LOGFILE=${BASE}/temp/${EMAIL}/${UNIQUEID}.txt\n   mkdir -p temp/${EMAIL} && cd $_\n-  echo -e \"Cloning Repository...\\n\"\n+  echo -e \"Cloning Repository...\\n\" | tee -a ${LOGFILE}\n   git clone -q https://:@${PLATFORM}/${USERNAME}/${REPONAME} \"$UNIQUEID\" >/dev/null 2>&1\n   if [ $? -ne 0 ]; then\n     >&2 echo -e \"Failed to Clone. Repository does not exist.\\n\"\n     exit 4\n   fi\n   cd ${UNIQUEID}\n-  echo -e \"Repository Cloned Successfully!\\n\"\n+  print_log \"Repository Cloned Successfully!\\n\"\n else\n   git clone -q https://github.com/fossasia/yaydoc.git yaydocclone\n   BASE=$(pwd)/yaydocclone\n fi\n \n+LOGFILE=${BASE}/temp/${EMAIL}/${UNIQUEID}.txt","path":"generate.sh","position":32,"original_position":32,"commit_id":"8b20358449e415351ed6cf046dc2c25ae5097431","original_commit_id":"8b20358449e415351ed6cf046dc2c25ae5097431","user":{"login":"pri22296","id":11555869,"avatar_url":"https://avatars0.githubusercontent.com/u/11555869?v=3","gravatar_id":"","url":"https://api.github.com/users/pri22296","html_url":"https://github.com/pri22296","followers_url":"https://api.github.com/users/pri22296/followers","following_url":"https://api.github.com/users/pri22296/following{/other_user}","gists_url":"https://api.github.com/users/pri22296/gists{/gist_id}","starred_url":"https://api.github.com/users/pri22296/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pri22296/subscriptions","organizations_url":"https://api.github.com/users/pri22296/orgs","repos_url":"https://api.github.com/users/pri22296/repos","events_url":"https://api.github.com/users/pri22296/events{/privacy}","received_events_url":"https://api.github.com/users/pri22296/received_events","type":"User","site_admin":false},"body":"Then we would also have to `cat` it on each exit point. No need for it. Instead just define LOGFILE in webUI and then in the print functions to the check if it is defined or not.","created_at":"2017-07-08T14:14:05Z","updated_at":"2017-07-08T14:14:06Z","html_url":"https://github.com/fossasia/yaydoc/pull/232#discussion_r126282874","pull_request_url":"https://api.github.com/repos/fossasia/yaydoc/pulls/232","_links":{"self":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/comments/126282874"},"html":{"href":"https://github.com/fossasia/yaydoc/pull/232#discussion_r126282874"},"pull_request":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/232"}}},"pull_request":{"url":"https://api.github.com/repos/fossasia/yaydoc/pulls/232","id":129476252,"html_url":"https://github.com/fossasia/yaydoc/pull/232","diff_url":"https://github.com/fossasia/yaydoc/pull/232.diff","patch_url":"https://github.com/fossasia/yaydoc/pull/232.patch","issue_url":"https://api.github.com/repos/fossasia/yaydoc/issues/232","number":232,"state":"open","locked":false,"title":"Fixes #228 Display basic logs and store detailed logs for retrieval","user":{"login":"imujjwal96","id":8245662,"avatar_url":"https://avatars3.githubusercontent.com/u/8245662?v=3","gravatar_id":"","url":"https://api.github.com/users/imujjwal96","html_url":"https://github.com/imujjwal96","followers_url":"https://api.github.com/users/imujjwal96/followers","following_url":"https://api.github.com/users/imujjwal96/following{/other_user}","gists_url":"https://api.github.com/users/imujjwal96/gists{/gist_id}","starred_url":"https://api.github.com/users/imujjwal96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imujjwal96/subscriptions","organizations_url":"https://api.github.com/users/imujjwal96/orgs","repos_url":"https://api.github.com/users/imujjwal96/repos","events_url":"https://api.github.com/users/imujjwal96/events{/privacy}","received_events_url":"https://api.github.com/users/imujjwal96/received_events","type":"User","site_admin":false},"body":"<!--\r\nThanks for submitting a change to yaydoc!\r\n\r\nTo make it possible for us to get your change reviewed and merged please fill out below information carefully.\r\n\r\n-->\r\n\r\n## Description\r\n<!--- Describe your changes in detail -->\r\nThe detailed logs have been stored in a file `${UNIQUEID}.txt` and are displayed on user's demand in a Modal. Otherwise only basic logs are shown without technical jargon\r\n## Related Issue\r\n<!--- This project only accepts pull requests related to open issues -->\r\n<!--- If suggesting a new feature or change, please discuss it in an issue first -->\r\n<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->\r\n<!--- Please link to the issue here: -->\r\nissue: #228 \r\n\r\n## How Has This Been Tested?\r\n<!--- Please describe in detail how you tested your changes. -->\r\n<!--- Include details of your testing environment, and the tests you ran to -->\r\n<!--- see how your change affects other areas of the code, etc. -->\r\nCan be tested at: https://yaydoc6.herokuapp.com/\r\n\r\n## Screenshots (if appropriate):\r\n![image](https://user-images.githubusercontent.com/8245662/27974507-db96fa20-637b-11e7-95aa-93d9f5c0db5b.png)\r\n![image](https://user-images.githubusercontent.com/8245662/27974536-edc98c9e-637b-11e7-8430-5baa8042e6b6.png)\r\n\r\n## Types of changes\r\n<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->\r\n- [ ] Bug fix \r\n- [x] New feature\r\n\r\n## Checklist:\r\n<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->\r\n- [x] I have read the [Contribution & Best practices Guide](https://blog.fossasia.org/open-source-developer-guide-and-best-practices-at-fossasia) and my PR follows them.\r\n- [x] There is a corresponding issue for this pull request.\r\n- [x] Mentioned the Issue number in the pull request commit message `Fixes #<number> commit message`\r\n- [x] There is only one commit per issue.\r\n","created_at":"2017-07-07T19:54:40Z","updated_at":"2017-07-08T14:14:06Z","closed_at":null,"merged_at":null,"merge_commit_sha":"cfd7dbc485c4141f197de38b60111c450123aa81","assignee":{"login":"imujjwal96","id":8245662,"avatar_url":"https://avatars3.githubusercontent.com/u/8245662?v=3","gravatar_id":"","url":"https://api.github.com/users/imujjwal96","html_url":"https://github.com/imujjwal96","followers_url":"https://api.github.com/users/imujjwal96/followers","following_url":"https://api.github.com/users/imujjwal96/following{/other_user}","gists_url":"https://api.github.com/users/imujjwal96/gists{/gist_id}","starred_url":"https://api.github.com/users/imujjwal96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imujjwal96/subscriptions","organizations_url":"https://api.github.com/users/imujjwal96/orgs","repos_url":"https://api.github.com/users/imujjwal96/repos","events_url":"https://api.github.com/users/imujjwal96/events{/privacy}","received_events_url":"https://api.github.com/users/imujjwal96/received_events","type":"User","site_admin":false},"assignees":[{"login":"imujjwal96","id":8245662,"avatar_url":"https://avatars3.githubusercontent.com/u/8245662?v=3","gravatar_id":"","url":"https://api.github.com/users/imujjwal96","html_url":"https://github.com/imujjwal96","followers_url":"https://api.github.com/users/imujjwal96/followers","following_url":"https://api.github.com/users/imujjwal96/following{/other_user}","gists_url":"https://api.github.com/users/imujjwal96/gists{/gist_id}","starred_url":"https://api.github.com/users/imujjwal96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imujjwal96/subscriptions","organizations_url":"https://api.github.com/users/imujjwal96/orgs","repos_url":"https://api.github.com/users/imujjwal96/repos","events_url":"https://api.github.com/users/imujjwal96/events{/privacy}","received_events_url":"https://api.github.com/users/imujjwal96/received_events","type":"User","site_admin":false}],"requested_reviewers":[{"login":"sch00lb0y","id":10010900,"avatar_url":"https://avatars0.githubusercontent.com/u/10010900?v=3","gravatar_id":"","url":"https://api.github.com/users/sch00lb0y","html_url":"https://github.com/sch00lb0y","followers_url":"https://api.github.com/users/sch00lb0y/followers","following_url":"https://api.github.com/users/sch00lb0y/following{/other_user}","gists_url":"https://api.github.com/users/sch00lb0y/gists{/gist_id}","starred_url":"https://api.github.com/users/sch00lb0y/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sch00lb0y/subscriptions","organizations_url":"https://api.github.com/users/sch00lb0y/orgs","repos_url":"https://api.github.com/users/sch00lb0y/repos","events_url":"https://api.github.com/users/sch00lb0y/events{/privacy}","received_events_url":"https://api.github.com/users/sch00lb0y/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/fossasia/yaydoc/pulls/232/commits","review_comments_url":"https://api.github.com/repos/fossasia/yaydoc/pulls/232/comments","review_comment_url":"https://api.github.com/repos/fossasia/yaydoc/pulls/comments{/number}","comments_url":"https://api.github.com/repos/fossasia/yaydoc/issues/232/comments","statuses_url":"https://api.github.com/repos/fossasia/yaydoc/statuses/8b20358449e415351ed6cf046dc2c25ae5097431","head":{"label":"imujjwal96:issue228","ref":"issue228","sha":"8b20358449e415351ed6cf046dc2c25ae5097431","user":{"login":"imujjwal96","id":8245662,"avatar_url":"https://avatars3.githubusercontent.com/u/8245662?v=3","gravatar_id":"","url":"https://api.github.com/users/imujjwal96","html_url":"https://github.com/imujjwal96","followers_url":"https://api.github.com/users/imujjwal96/followers","following_url":"https://api.github.com/users/imujjwal96/following{/other_user}","gists_url":"https://api.github.com/users/imujjwal96/gists{/gist_id}","starred_url":"https://api.github.com/users/imujjwal96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imujjwal96/subscriptions","organizations_url":"https://api.github.com/users/imujjwal96/orgs","repos_url":"https://api.github.com/users/imujjwal96/repos","events_url":"https://api.github.com/users/imujjwal96/events{/privacy}","received_events_url":"https://api.github.com/users/imujjwal96/received_events","type":"User","site_admin":false},"repo":{"id":93080345,"name":"yaydoc","full_name":"imujjwal96/yaydoc","owner":{"login":"imujjwal96","id":8245662,"avatar_url":"https://avatars3.githubusercontent.com/u/8245662?v=3","gravatar_id":"","url":"https://api.github.com/users/imujjwal96","html_url":"https://github.com/imujjwal96","followers_url":"https://api.github.com/users/imujjwal96/followers","following_url":"https://api.github.com/users/imujjwal96/following{/other_user}","gists_url":"https://api.github.com/users/imujjwal96/gists{/gist_id}","starred_url":"https://api.github.com/users/imujjwal96/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imujjwal96/subscriptions","organizations_url":"https://api.github.com/users/imujjwal96/orgs","repos_url":"https://api.github.com/users/imujjwal96/repos","events_url":"https://api.github.com/users/imujjwal96/events{/privacy}","received_events_url":"https://api.github.com/users/imujjwal96/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/imujjwal96/yaydoc","description":"Docs! Yay!","fork":true,"url":"https://api.github.com/repos/imujjwal96/yaydoc","forks_url":"https://api.github.com/repos/imujjwal96/yaydoc/forks","keys_url":"https://api.github.com/repos/imujjwal96/yaydoc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/imujjwal96/yaydoc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/imujjwal96/yaydoc/teams","hooks_url":"https://api.github.com/repos/imujjwal96/yaydoc/hooks","issue_events_url":"https://api.github.com/repos/imujjwal96/yaydoc/issues/events{/number}","events_url":"https://api.github.com/repos/imujjwal96/yaydoc/events","assignees_url":"https://api.github.com/repos/imujjwal96/yaydoc/assignees{/user}","branches_url":"https://api.github.com/repos/imujjwal96/yaydoc/branches{/branch}","tags_url":"https://api.github.com/repos/imujjwal96/yaydoc/tags","blobs_url":"https://api.github.com/repos/imujjwal96/yaydoc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/imujjwal96/yaydoc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/imujjwal96/yaydoc/git/refs{/sha}","trees_url":"https://api.github.com/repos/imujjwal96/yaydoc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/imujjwal96/yaydoc/statuses/{sha}","languages_url":"https://api.github.com/repos/imujjwal96/yaydoc/languages","stargazers_url":"https://api.github.com/repos/imujjwal96/yaydoc/stargazers","contributors_url":"https://api.github.com/repos/imujjwal96/yaydoc/contributors","subscribers_url":"https://api.github.com/repos/imujjwal96/yaydoc/subscribers","subscription_url":"https://api.github.com/repos/imujjwal96/yaydoc/subscription","commits_url":"https://api.github.com/repos/imujjwal96/yaydoc/commits{/sha}","git_commits_url":"https://api.github.com/repos/imujjwal96/yaydoc/git/commits{/sha}","comments_url":"https://api.github.com/repos/imujjwal96/yaydoc/comments{/number}","issue_comment_url":"https://api.github.com/repos/imujjwal96/yaydoc/issues/comments{/number}","contents_url":"https://api.github.com/repos/imujjwal96/yaydoc/contents/{+path}","compare_url":"https://api.github.com/repos/imujjwal96/yaydoc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/imujjwal96/yaydoc/merges","archive_url":"https://api.github.com/repos/imujjwal96/yaydoc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/imujjwal96/yaydoc/downloads","issues_url":"https://api.github.com/repos/imujjwal96/yaydoc/issues{/number}","pulls_url":"https://api.github.com/repos/imujjwal96/yaydoc/pulls{/number}","milestones_url":"https://api.github.com/repos/imujjwal96/yaydoc/milestones{/number}","notifications_url":"https://api.github.com/repos/imujjwal96/yaydoc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/imujjwal96/yaydoc/labels{/name}","releases_url":"https://api.github.com/repos/imujjwal96/yaydoc/releases{/id}","deployments_url":"https://api.github.com/repos/imujjwal96/yaydoc/deployments","created_at":"2017-06-01T17:01:17Z","updated_at":"2017-06-01T17:01:18Z","pushed_at":"2017-07-08T11:33:01Z","git_url":"git://github.com/imujjwal96/yaydoc.git","ssh_url":"git@github.com:imujjwal96/yaydoc.git","clone_url":"https://github.com/imujjwal96/yaydoc.git","svn_url":"https://github.com/imujjwal96/yaydoc","homepage":null,"size":508,"stargazers_count":0,"watchers_count":0,"language":"CSS","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"fossasia:master","ref":"master","sha":"8f444dfa7dc7819b8996ba7231c0a12696b0f456","user":{"login":"fossasia","id":6295529,"avatar_url":"https://avatars0.githubusercontent.com/u/6295529?v=3","gravatar_id":"","url":"https://api.github.com/users/fossasia","html_url":"https://github.com/fossasia","followers_url":"https://api.github.com/users/fossasia/followers","following_url":"https://api.github.com/users/fossasia/following{/other_user}","gists_url":"https://api.github.com/users/fossasia/gists{/gist_id}","starred_url":"https://api.github.com/users/fossasia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fossasia/subscriptions","organizations_url":"https://api.github.com/users/fossasia/orgs","repos_url":"https://api.github.com/users/fossasia/repos","events_url":"https://api.github.com/users/fossasia/events{/privacy}","received_events_url":"https://api.github.com/users/fossasia/received_events","type":"Organization","site_admin":false},"repo":{"id":90524814,"name":"yaydoc","full_name":"fossasia/yaydoc","owner":{"login":"fossasia","id":6295529,"avatar_url":"https://avatars0.githubusercontent.com/u/6295529?v=3","gravatar_id":"","url":"https://api.github.com/users/fossasia","html_url":"https://github.com/fossasia","followers_url":"https://api.github.com/users/fossasia/followers","following_url":"https://api.github.com/users/fossasia/following{/other_user}","gists_url":"https://api.github.com/users/fossasia/gists{/gist_id}","starred_url":"https://api.github.com/users/fossasia/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fossasia/subscriptions","organizations_url":"https://api.github.com/users/fossasia/orgs","repos_url":"https://api.github.com/users/fossasia/repos","events_url":"https://api.github.com/users/fossasia/events{/privacy}","received_events_url":"https://api.github.com/users/fossasia/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/fossasia/yaydoc","description":"Docs! Yay! https://yaydoc.herokuapp.com","fork":false,"url":"https://api.github.com/repos/fossasia/yaydoc","forks_url":"https://api.github.com/repos/fossasia/yaydoc/forks","keys_url":"https://api.github.com/repos/fossasia/yaydoc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fossasia/yaydoc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fossasia/yaydoc/teams","hooks_url":"https://api.github.com/repos/fossasia/yaydoc/hooks","issue_events_url":"https://api.github.com/repos/fossasia/yaydoc/issues/events{/number}","events_url":"https://api.github.com/repos/fossasia/yaydoc/events","assignees_url":"https://api.github.com/repos/fossasia/yaydoc/assignees{/user}","branches_url":"https://api.github.com/repos/fossasia/yaydoc/branches{/branch}","tags_url":"https://api.github.com/repos/fossasia/yaydoc/tags","blobs_url":"https://api.github.com/repos/fossasia/yaydoc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fossasia/yaydoc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fossasia/yaydoc/git/refs{/sha}","trees_url":"https://api.github.com/repos/fossasia/yaydoc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fossasia/yaydoc/statuses/{sha}","languages_url":"https://api.github.com/repos/fossasia/yaydoc/languages","stargazers_url":"https://api.github.com/repos/fossasia/yaydoc/stargazers","contributors_url":"https://api.github.com/repos/fossasia/yaydoc/contributors","subscribers_url":"https://api.github.com/repos/fossasia/yaydoc/subscribers","subscription_url":"https://api.github.com/repos/fossasia/yaydoc/subscription","commits_url":"https://api.github.com/repos/fossasia/yaydoc/commits{/sha}","git_commits_url":"https://api.github.com/repos/fossasia/yaydoc/git/commits{/sha}","comments_url":"https://api.github.com/repos/fossasia/yaydoc/comments{/number}","issue_comment_url":"https://api.github.com/repos/fossasia/yaydoc/issues/comments{/number}","contents_url":"https://api.github.com/repos/fossasia/yaydoc/contents/{+path}","compare_url":"https://api.github.com/repos/fossasia/yaydoc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fossasia/yaydoc/merges","archive_url":"https://api.github.com/repos/fossasia/yaydoc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fossasia/yaydoc/downloads","issues_url":"https://api.github.com/repos/fossasia/yaydoc/issues{/number}","pulls_url":"https://api.github.com/repos/fossasia/yaydoc/pulls{/number}","milestones_url":"https://api.github.com/repos/fossasia/yaydoc/milestones{/number}","notifications_url":"https://api.github.com/repos/fossasia/yaydoc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fossasia/yaydoc/labels{/name}","releases_url":"https://api.github.com/repos/fossasia/yaydoc/releases{/id}","deployments_url":"https://api.github.com/repos/fossasia/yaydoc/deployments","created_at":"2017-05-07T10:26:43Z","updated_at":"2017-07-08T10:04:03Z","pushed_at":"2017-07-08T14:11:27Z","git_url":"git://github.com/fossasia/yaydoc.git","ssh_url":"git@github.com:fossasia/yaydoc.git","clone_url":"https://github.com/fossasia/yaydoc.git","svn_url":"https://github.com/fossasia/yaydoc","homepage":"http://yaydoc.fossasia.org","size":661,"stargazers_count":6,"watchers_count":6,"language":"CSS","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":4,"mirror_url":null,"open_issues_count":24,"forks":4,"open_issues":24,"watchers":6,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/232"},"html":{"href":"https://github.com/fossasia/yaydoc/pull/232"},"issue":{"href":"https://api.github.com/repos/fossasia/yaydoc/issues/232"},"comments":{"href":"https://api.github.com/repos/fossasia/yaydoc/issues/232/comments"},"review_comments":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/232/comments"},"review_comment":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/fossasia/yaydoc/pulls/232/commits"},"statuses":{"href":"https://api.github.com/repos/fossasia/yaydoc/statuses/8b20358449e415351ed6cf046dc2c25ae5097431"}}}}
{"actor":{"display_login":"pri22296"}}