{"action":"created","comment":{"url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/comments/185696200","pull_request_review_id":117136052,"id":185696200,"diff_hunk":"@@ -48,6 +48,7 @@ def __init__(self, _days, _filename, _username):\n         self.root = Tk.Tk()\r\n         self.filename = _filename\r\n         self.username = _username\r\n+        self.logname = _logname\r","path":"prog-o-meter.py","position":13,"original_position":13,"commit_id":"1cc6f778fe3b361c7e26d5e27a0cf206eebf02ce","original_commit_id":"dae341a9c1b065af27f5dcaa8c2624208462d09e","user":{"login":"sdkwok","id":26441890,"avatar_url":"https://avatars3.githubusercontent.com/u/26441890?v=4","gravatar_id":"","url":"https://api.github.com/users/sdkwok","html_url":"https://github.com/sdkwok","followers_url":"https://api.github.com/users/sdkwok/followers","following_url":"https://api.github.com/users/sdkwok/following{/other_user}","gists_url":"https://api.github.com/users/sdkwok/gists{/gist_id}","starred_url":"https://api.github.com/users/sdkwok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdkwok/subscriptions","organizations_url":"https://api.github.com/users/sdkwok/orgs","repos_url":"https://api.github.com/users/sdkwok/repos","events_url":"https://api.github.com/users/sdkwok/events{/privacy}","received_events_url":"https://api.github.com/users/sdkwok/received_events","type":"User","site_admin":false},"body":"The logname attribute is used inside the function log_entry: inside this function, on line 179, we pass self.logname to the function update_log_file, which updates the log file with the text entered by the user.","created_at":"2018-05-03T04:43:17Z","updated_at":"2018-05-03T04:43:17Z","html_url":"https://github.com/lineaba/prog-o-meter/pull/70#discussion_r185696200","pull_request_url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/comments/185696200"},"html":{"href":"https://github.com/lineaba/prog-o-meter/pull/70#discussion_r185696200"},"pull_request":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70"}},"in_reply_to_id":185691798},"pull_request":{"url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70","id":185607263,"html_url":"https://github.com/lineaba/prog-o-meter/pull/70","diff_url":"https://github.com/lineaba/prog-o-meter/pull/70.diff","patch_url":"https://github.com/lineaba/prog-o-meter/pull/70.patch","issue_url":"https://api.github.com/repos/lineaba/prog-o-meter/issues/70","number":70,"state":"open","locked":false,"title":"Added log feature","user":{"login":"sdkwok","id":26441890,"avatar_url":"https://avatars3.githubusercontent.com/u/26441890?v=4","gravatar_id":"","url":"https://api.github.com/users/sdkwok","html_url":"https://github.com/sdkwok","followers_url":"https://api.github.com/users/sdkwok/followers","following_url":"https://api.github.com/users/sdkwok/following{/other_user}","gists_url":"https://api.github.com/users/sdkwok/gists{/gist_id}","starred_url":"https://api.github.com/users/sdkwok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdkwok/subscriptions","organizations_url":"https://api.github.com/users/sdkwok/orgs","repos_url":"https://api.github.com/users/sdkwok/repos","events_url":"https://api.github.com/users/sdkwok/events{/privacy}","received_events_url":"https://api.github.com/users/sdkwok/received_events","type":"User","site_admin":false},"body":"__What issue are you solving? (Give #issuenumber)__\r\n\r\n#32 (Add a logging feature)\r\n\r\n__How do you solve it? (please write this so a human can understand it - bulletpoints are okay)__\r\n\r\nMade the following changes.\r\n\r\nProgressGUI class:\r\n\r\n--added logname attribute to the class\r\n--added a button with the text \"Make log entry\" to button_layout which calls the function log_entry\r\n--added function log_entry which opens a window with text box to allow the user to make a log entry. Window has three buttons: \"save\" to save to the log by calling the update_log_file function, \"clear\" to clear text in the text box, and \"close\" to close the window\r\n\r\nUsernameGUI class:\r\n\r\n--added a function update_log_file which appends a log entry to a given log file, placing a timestamp above the entry\r\n--edited the main() function so that it automatically creates an empty log file for new users. This file has the name \"[username]_log.txt\"\r\n\r\n__If there is something specific in your solution, that you would like the reviewer to give feedback on, tell us here:__\r\n\r\nI changed the main routine so that a new user now automatically has an empty log file created for them. I thought this was the easiest way, but I am open to other suggestions.\r\n\r\nA user can update the log as many times as he/she wishes. Again, I am open to other suggestions.\r\n\r\n_If this is your first pull request to the prog-o-meter project, we would really appreciate it if you would fill out this Google form.\r\nThis lets us know more about our contributors, and what we can do to make it easier and nicer to contribute to the prog-o-meter project.\r\nPlease submit your answer [here](https://docs.google.com/forms/d/e/1FAIpQLSdQqs9shjcyJTTqbkKjGv85Irxxaljvw0H357zuyrRwXq9_nA/viewform?usp=sf_link) and put an 'x' in the box below._\r\n\r\n- [ X] It is my first pull request to prog-o-meter, and I have answered the survey","created_at":"2018-05-03T02:56:59Z","updated_at":"2018-05-03T04:43:17Z","closed_at":null,"merged_at":null,"merge_commit_sha":"36d13a76a7fcd4a0f85a595cbb195fd21854d2b5","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70/commits","review_comments_url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70/comments","review_comment_url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/comments{/number}","comments_url":"https://api.github.com/repos/lineaba/prog-o-meter/issues/70/comments","statuses_url":"https://api.github.com/repos/lineaba/prog-o-meter/statuses/1cc6f778fe3b361c7e26d5e27a0cf206eebf02ce","head":{"label":"sdkwok:master","ref":"master","sha":"1cc6f778fe3b361c7e26d5e27a0cf206eebf02ce","user":{"login":"sdkwok","id":26441890,"avatar_url":"https://avatars3.githubusercontent.com/u/26441890?v=4","gravatar_id":"","url":"https://api.github.com/users/sdkwok","html_url":"https://github.com/sdkwok","followers_url":"https://api.github.com/users/sdkwok/followers","following_url":"https://api.github.com/users/sdkwok/following{/other_user}","gists_url":"https://api.github.com/users/sdkwok/gists{/gist_id}","starred_url":"https://api.github.com/users/sdkwok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdkwok/subscriptions","organizations_url":"https://api.github.com/users/sdkwok/orgs","repos_url":"https://api.github.com/users/sdkwok/repos","events_url":"https://api.github.com/users/sdkwok/events{/privacy}","received_events_url":"https://api.github.com/users/sdkwok/received_events","type":"User","site_admin":false},"repo":{"id":131929987,"name":"prog-o-meter","full_name":"sdkwok/prog-o-meter","owner":{"login":"sdkwok","id":26441890,"avatar_url":"https://avatars3.githubusercontent.com/u/26441890?v=4","gravatar_id":"","url":"https://api.github.com/users/sdkwok","html_url":"https://github.com/sdkwok","followers_url":"https://api.github.com/users/sdkwok/followers","following_url":"https://api.github.com/users/sdkwok/following{/other_user}","gists_url":"https://api.github.com/users/sdkwok/gists{/gist_id}","starred_url":"https://api.github.com/users/sdkwok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sdkwok/subscriptions","organizations_url":"https://api.github.com/users/sdkwok/orgs","repos_url":"https://api.github.com/users/sdkwok/repos","events_url":"https://api.github.com/users/sdkwok/events{/privacy}","received_events_url":"https://api.github.com/users/sdkwok/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/sdkwok/prog-o-meter","description":"The prog-o-meter is for tracking your progress during a #100DaysofCode challenge (or any other #100DaysofX challenge) Made for newbies by newbies. Everyone is more than welcome to contribute. Project is for learning workflow and team cooperation in software development","fork":true,"url":"https://api.github.com/repos/sdkwok/prog-o-meter","forks_url":"https://api.github.com/repos/sdkwok/prog-o-meter/forks","keys_url":"https://api.github.com/repos/sdkwok/prog-o-meter/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sdkwok/prog-o-meter/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sdkwok/prog-o-meter/teams","hooks_url":"https://api.github.com/repos/sdkwok/prog-o-meter/hooks","issue_events_url":"https://api.github.com/repos/sdkwok/prog-o-meter/issues/events{/number}","events_url":"https://api.github.com/repos/sdkwok/prog-o-meter/events","assignees_url":"https://api.github.com/repos/sdkwok/prog-o-meter/assignees{/user}","branches_url":"https://api.github.com/repos/sdkwok/prog-o-meter/branches{/branch}","tags_url":"https://api.github.com/repos/sdkwok/prog-o-meter/tags","blobs_url":"https://api.github.com/repos/sdkwok/prog-o-meter/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sdkwok/prog-o-meter/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sdkwok/prog-o-meter/git/refs{/sha}","trees_url":"https://api.github.com/repos/sdkwok/prog-o-meter/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sdkwok/prog-o-meter/statuses/{sha}","languages_url":"https://api.github.com/repos/sdkwok/prog-o-meter/languages","stargazers_url":"https://api.github.com/repos/sdkwok/prog-o-meter/stargazers","contributors_url":"https://api.github.com/repos/sdkwok/prog-o-meter/contributors","subscribers_url":"https://api.github.com/repos/sdkwok/prog-o-meter/subscribers","subscription_url":"https://api.github.com/repos/sdkwok/prog-o-meter/subscription","commits_url":"https://api.github.com/repos/sdkwok/prog-o-meter/commits{/sha}","git_commits_url":"https://api.github.com/repos/sdkwok/prog-o-meter/git/commits{/sha}","comments_url":"https://api.github.com/repos/sdkwok/prog-o-meter/comments{/number}","issue_comment_url":"https://api.github.com/repos/sdkwok/prog-o-meter/issues/comments{/number}","contents_url":"https://api.github.com/repos/sdkwok/prog-o-meter/contents/{+path}","compare_url":"https://api.github.com/repos/sdkwok/prog-o-meter/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sdkwok/prog-o-meter/merges","archive_url":"https://api.github.com/repos/sdkwok/prog-o-meter/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sdkwok/prog-o-meter/downloads","issues_url":"https://api.github.com/repos/sdkwok/prog-o-meter/issues{/number}","pulls_url":"https://api.github.com/repos/sdkwok/prog-o-meter/pulls{/number}","milestones_url":"https://api.github.com/repos/sdkwok/prog-o-meter/milestones{/number}","notifications_url":"https://api.github.com/repos/sdkwok/prog-o-meter/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sdkwok/prog-o-meter/labels{/name}","releases_url":"https://api.github.com/repos/sdkwok/prog-o-meter/releases{/id}","deployments_url":"https://api.github.com/repos/sdkwok/prog-o-meter/deployments","created_at":"2018-05-03T02:19:43Z","updated_at":"2018-05-03T04:42:27Z","pushed_at":"2018-05-03T04:42:25Z","git_url":"git://github.com/sdkwok/prog-o-meter.git","ssh_url":"git@github.com:sdkwok/prog-o-meter.git","clone_url":"https://github.com/sdkwok/prog-o-meter.git","svn_url":"https://github.com/sdkwok/prog-o-meter","homepage":"","size":70,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit"},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"lineaba:master","ref":"master","sha":"3738c9e9f6b4d5a02222222bf55389c098be264f","user":{"login":"lineaba","id":32916783,"avatar_url":"https://avatars0.githubusercontent.com/u/32916783?v=4","gravatar_id":"","url":"https://api.github.com/users/lineaba","html_url":"https://github.com/lineaba","followers_url":"https://api.github.com/users/lineaba/followers","following_url":"https://api.github.com/users/lineaba/following{/other_user}","gists_url":"https://api.github.com/users/lineaba/gists{/gist_id}","starred_url":"https://api.github.com/users/lineaba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lineaba/subscriptions","organizations_url":"https://api.github.com/users/lineaba/orgs","repos_url":"https://api.github.com/users/lineaba/repos","events_url":"https://api.github.com/users/lineaba/events{/privacy}","received_events_url":"https://api.github.com/users/lineaba/received_events","type":"User","site_admin":false},"repo":{"id":130426390,"name":"prog-o-meter","full_name":"lineaba/prog-o-meter","owner":{"login":"lineaba","id":32916783,"avatar_url":"https://avatars0.githubusercontent.com/u/32916783?v=4","gravatar_id":"","url":"https://api.github.com/users/lineaba","html_url":"https://github.com/lineaba","followers_url":"https://api.github.com/users/lineaba/followers","following_url":"https://api.github.com/users/lineaba/following{/other_user}","gists_url":"https://api.github.com/users/lineaba/gists{/gist_id}","starred_url":"https://api.github.com/users/lineaba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lineaba/subscriptions","organizations_url":"https://api.github.com/users/lineaba/orgs","repos_url":"https://api.github.com/users/lineaba/repos","events_url":"https://api.github.com/users/lineaba/events{/privacy}","received_events_url":"https://api.github.com/users/lineaba/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/lineaba/prog-o-meter","description":"The prog-o-meter is for tracking your progress during a #100DaysofCode challenge (or any other #100DaysofX challenge) Made for newbies by newbies. Everyone is more than welcome to contribute. Project is for learning workflow and team cooperation in software development","fork":false,"url":"https://api.github.com/repos/lineaba/prog-o-meter","forks_url":"https://api.github.com/repos/lineaba/prog-o-meter/forks","keys_url":"https://api.github.com/repos/lineaba/prog-o-meter/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lineaba/prog-o-meter/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lineaba/prog-o-meter/teams","hooks_url":"https://api.github.com/repos/lineaba/prog-o-meter/hooks","issue_events_url":"https://api.github.com/repos/lineaba/prog-o-meter/issues/events{/number}","events_url":"https://api.github.com/repos/lineaba/prog-o-meter/events","assignees_url":"https://api.github.com/repos/lineaba/prog-o-meter/assignees{/user}","branches_url":"https://api.github.com/repos/lineaba/prog-o-meter/branches{/branch}","tags_url":"https://api.github.com/repos/lineaba/prog-o-meter/tags","blobs_url":"https://api.github.com/repos/lineaba/prog-o-meter/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lineaba/prog-o-meter/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lineaba/prog-o-meter/git/refs{/sha}","trees_url":"https://api.github.com/repos/lineaba/prog-o-meter/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lineaba/prog-o-meter/statuses/{sha}","languages_url":"https://api.github.com/repos/lineaba/prog-o-meter/languages","stargazers_url":"https://api.github.com/repos/lineaba/prog-o-meter/stargazers","contributors_url":"https://api.github.com/repos/lineaba/prog-o-meter/contributors","subscribers_url":"https://api.github.com/repos/lineaba/prog-o-meter/subscribers","subscription_url":"https://api.github.com/repos/lineaba/prog-o-meter/subscription","commits_url":"https://api.github.com/repos/lineaba/prog-o-meter/commits{/sha}","git_commits_url":"https://api.github.com/repos/lineaba/prog-o-meter/git/commits{/sha}","comments_url":"https://api.github.com/repos/lineaba/prog-o-meter/comments{/number}","issue_comment_url":"https://api.github.com/repos/lineaba/prog-o-meter/issues/comments{/number}","contents_url":"https://api.github.com/repos/lineaba/prog-o-meter/contents/{+path}","compare_url":"https://api.github.com/repos/lineaba/prog-o-meter/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lineaba/prog-o-meter/merges","archive_url":"https://api.github.com/repos/lineaba/prog-o-meter/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lineaba/prog-o-meter/downloads","issues_url":"https://api.github.com/repos/lineaba/prog-o-meter/issues{/number}","pulls_url":"https://api.github.com/repos/lineaba/prog-o-meter/pulls{/number}","milestones_url":"https://api.github.com/repos/lineaba/prog-o-meter/milestones{/number}","notifications_url":"https://api.github.com/repos/lineaba/prog-o-meter/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lineaba/prog-o-meter/labels{/name}","releases_url":"https://api.github.com/repos/lineaba/prog-o-meter/releases{/id}","deployments_url":"https://api.github.com/repos/lineaba/prog-o-meter/deployments","created_at":"2018-04-21T00:41:15Z","updated_at":"2018-05-02T20:23:21Z","pushed_at":"2018-05-03T04:42:27Z","git_url":"git://github.com/lineaba/prog-o-meter.git","ssh_url":"git@github.com:lineaba/prog-o-meter.git","clone_url":"https://github.com/lineaba/prog-o-meter.git","svn_url":"https://github.com/lineaba/prog-o-meter","homepage":"","size":63,"stargazers_count":7,"watchers_count":7,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":11,"mirror_url":null,"archived":false,"open_issues_count":32,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit"},"forks":11,"open_issues":32,"watchers":7,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70"},"html":{"href":"https://github.com/lineaba/prog-o-meter/pull/70"},"issue":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/issues/70"},"comments":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/issues/70/comments"},"review_comments":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70/comments"},"review_comment":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/pulls/70/commits"},"statuses":{"href":"https://api.github.com/repos/lineaba/prog-o-meter/statuses/1cc6f778fe3b361c7e26d5e27a0cf206eebf02ce"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"sdkwok"}}