{"action":"created","comment":{"url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/comments/45090526","id":45090526,"diff_hunk":"@@ -22,6 +22,11 @@\n       <td id=\"task_link_cell\">","path":"task-list/app/views/tasks/index.html.erb","position":18,"original_position":18,"commit_id":"f0b3649ba36f2220c88dd735e94baf2057591e55","original_commit_id":"f0b3649ba36f2220c88dd735e94baf2057591e55","user":{"login":"Hamled","id":66355,"avatar_url":"https://avatars.githubusercontent.com/u/66355?v=3","gravatar_id":"","url":"https://api.github.com/users/Hamled","html_url":"https://github.com/Hamled","followers_url":"https://api.github.com/users/Hamled/followers","following_url":"https://api.github.com/users/Hamled/following{/other_user}","gists_url":"https://api.github.com/users/Hamled/gists{/gist_id}","starred_url":"https://api.github.com/users/Hamled/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Hamled/subscriptions","organizations_url":"https://api.github.com/users/Hamled/orgs","repos_url":"https://api.github.com/users/Hamled/repos","events_url":"https://api.github.com/users/Hamled/events{/privacy}","received_events_url":"https://api.github.com/users/Hamled/received_events","type":"User","site_admin":false},"body":"I don't think it is a good idea to use the `id` attribute here and in the following `td` tags. Because this code is within an `each` loop from line 13, it will be repeated for every element in `@tasks`, but we should only have a single tag with a given `id` value.\r\n\r\nI'd recommend either changing these attributes to `class`, or to use ERB tags to generate a unique ID like: `<td id=\"task_link_cell_<%= task.id %>\">`.","created_at":"2015-11-17T17:28:55Z","updated_at":"2015-11-17T17:28:55Z","html_url":"https://github.com/Ada-C4/TaskListRails/pull/90#discussion_r45090526","pull_request_url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90","_links":{"self":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/comments/45090526"},"html":{"href":"https://github.com/Ada-C4/TaskListRails/pull/90#discussion_r45090526"},"pull_request":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90"}}},"pull_request":{"url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90","id":50881798,"html_url":"https://github.com/Ada-C4/TaskListRails/pull/90","diff_url":"https://github.com/Ada-C4/TaskListRails/pull/90.diff","patch_url":"https://github.com/Ada-C4/TaskListRails/pull/90.patch","issue_url":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/90","number":90,"state":"open","locked":false,"title":"TaskListRails - Final Wave","user":{"login":"Kedevlin","id":13082148,"avatar_url":"https://avatars.githubusercontent.com/u/13082148?v=3","gravatar_id":"","url":"https://api.github.com/users/Kedevlin","html_url":"https://github.com/Kedevlin","followers_url":"https://api.github.com/users/Kedevlin/followers","following_url":"https://api.github.com/users/Kedevlin/following{/other_user}","gists_url":"https://api.github.com/users/Kedevlin/gists{/gist_id}","starred_url":"https://api.github.com/users/Kedevlin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kedevlin/subscriptions","organizations_url":"https://api.github.com/users/Kedevlin/orgs","repos_url":"https://api.github.com/users/Kedevlin/repos","events_url":"https://api.github.com/users/Kedevlin/events{/privacy}","received_events_url":"https://api.github.com/users/Kedevlin/received_events","type":"User","site_admin":false},"body":"","created_at":"2015-11-16T23:19:56Z","updated_at":"2015-11-17T17:28:55Z","closed_at":null,"merged_at":null,"merge_commit_sha":"cb91b9a9d3e62098e72f1db116cbde12bd9b62f9","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90/commits","review_comments_url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90/comments","review_comment_url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/90/comments","statuses_url":"https://api.github.com/repos/Ada-C4/TaskListRails/statuses/f0b3649ba36f2220c88dd735e94baf2057591e55","head":{"label":"Kedevlin:KED/master","ref":"KED/master","sha":"f0b3649ba36f2220c88dd735e94baf2057591e55","user":{"login":"Kedevlin","id":13082148,"avatar_url":"https://avatars.githubusercontent.com/u/13082148?v=3","gravatar_id":"","url":"https://api.github.com/users/Kedevlin","html_url":"https://github.com/Kedevlin","followers_url":"https://api.github.com/users/Kedevlin/followers","following_url":"https://api.github.com/users/Kedevlin/following{/other_user}","gists_url":"https://api.github.com/users/Kedevlin/gists{/gist_id}","starred_url":"https://api.github.com/users/Kedevlin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kedevlin/subscriptions","organizations_url":"https://api.github.com/users/Kedevlin/orgs","repos_url":"https://api.github.com/users/Kedevlin/repos","events_url":"https://api.github.com/users/Kedevlin/events{/privacy}","received_events_url":"https://api.github.com/users/Kedevlin/received_events","type":"User","site_admin":false},"repo":{"id":45875211,"name":"TaskListRails","full_name":"Kedevlin/TaskListRails","owner":{"login":"Kedevlin","id":13082148,"avatar_url":"https://avatars.githubusercontent.com/u/13082148?v=3","gravatar_id":"","url":"https://api.github.com/users/Kedevlin","html_url":"https://github.com/Kedevlin","followers_url":"https://api.github.com/users/Kedevlin/followers","following_url":"https://api.github.com/users/Kedevlin/following{/other_user}","gists_url":"https://api.github.com/users/Kedevlin/gists{/gist_id}","starred_url":"https://api.github.com/users/Kedevlin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Kedevlin/subscriptions","organizations_url":"https://api.github.com/users/Kedevlin/orgs","repos_url":"https://api.github.com/users/Kedevlin/repos","events_url":"https://api.github.com/users/Kedevlin/events{/privacy}","received_events_url":"https://api.github.com/users/Kedevlin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/Kedevlin/TaskListRails","description":"","fork":true,"url":"https://api.github.com/repos/Kedevlin/TaskListRails","forks_url":"https://api.github.com/repos/Kedevlin/TaskListRails/forks","keys_url":"https://api.github.com/repos/Kedevlin/TaskListRails/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Kedevlin/TaskListRails/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Kedevlin/TaskListRails/teams","hooks_url":"https://api.github.com/repos/Kedevlin/TaskListRails/hooks","issue_events_url":"https://api.github.com/repos/Kedevlin/TaskListRails/issues/events{/number}","events_url":"https://api.github.com/repos/Kedevlin/TaskListRails/events","assignees_url":"https://api.github.com/repos/Kedevlin/TaskListRails/assignees{/user}","branches_url":"https://api.github.com/repos/Kedevlin/TaskListRails/branches{/branch}","tags_url":"https://api.github.com/repos/Kedevlin/TaskListRails/tags","blobs_url":"https://api.github.com/repos/Kedevlin/TaskListRails/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Kedevlin/TaskListRails/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Kedevlin/TaskListRails/git/refs{/sha}","trees_url":"https://api.github.com/repos/Kedevlin/TaskListRails/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Kedevlin/TaskListRails/statuses/{sha}","languages_url":"https://api.github.com/repos/Kedevlin/TaskListRails/languages","stargazers_url":"https://api.github.com/repos/Kedevlin/TaskListRails/stargazers","contributors_url":"https://api.github.com/repos/Kedevlin/TaskListRails/contributors","subscribers_url":"https://api.github.com/repos/Kedevlin/TaskListRails/subscribers","subscription_url":"https://api.github.com/repos/Kedevlin/TaskListRails/subscription","commits_url":"https://api.github.com/repos/Kedevlin/TaskListRails/commits{/sha}","git_commits_url":"https://api.github.com/repos/Kedevlin/TaskListRails/git/commits{/sha}","comments_url":"https://api.github.com/repos/Kedevlin/TaskListRails/comments{/number}","issue_comment_url":"https://api.github.com/repos/Kedevlin/TaskListRails/issues/comments{/number}","contents_url":"https://api.github.com/repos/Kedevlin/TaskListRails/contents/{+path}","compare_url":"https://api.github.com/repos/Kedevlin/TaskListRails/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Kedevlin/TaskListRails/merges","archive_url":"https://api.github.com/repos/Kedevlin/TaskListRails/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Kedevlin/TaskListRails/downloads","issues_url":"https://api.github.com/repos/Kedevlin/TaskListRails/issues{/number}","pulls_url":"https://api.github.com/repos/Kedevlin/TaskListRails/pulls{/number}","milestones_url":"https://api.github.com/repos/Kedevlin/TaskListRails/milestones{/number}","notifications_url":"https://api.github.com/repos/Kedevlin/TaskListRails/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Kedevlin/TaskListRails/labels{/name}","releases_url":"https://api.github.com/repos/Kedevlin/TaskListRails/releases{/id}","created_at":"2015-11-09T23:58:16Z","updated_at":"2015-11-09T23:58:17Z","pushed_at":"2015-11-16T23:11:43Z","git_url":"git://github.com/Kedevlin/TaskListRails.git","ssh_url":"git@github.com:Kedevlin/TaskListRails.git","clone_url":"https://github.com/Kedevlin/TaskListRails.git","svn_url":"https://github.com/Kedevlin/TaskListRails","homepage":null,"size":59,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":true,"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":"Ada-C4:KED/master","ref":"KED/master","sha":"9fab384f608ca9e53974f99c44be1962962ab0a5","user":{"login":"Ada-C4","id":14096372,"avatar_url":"https://avatars.githubusercontent.com/u/14096372?v=3","gravatar_id":"","url":"https://api.github.com/users/Ada-C4","html_url":"https://github.com/Ada-C4","followers_url":"https://api.github.com/users/Ada-C4/followers","following_url":"https://api.github.com/users/Ada-C4/following{/other_user}","gists_url":"https://api.github.com/users/Ada-C4/gists{/gist_id}","starred_url":"https://api.github.com/users/Ada-C4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ada-C4/subscriptions","organizations_url":"https://api.github.com/users/Ada-C4/orgs","repos_url":"https://api.github.com/users/Ada-C4/repos","events_url":"https://api.github.com/users/Ada-C4/events{/privacy}","received_events_url":"https://api.github.com/users/Ada-C4/received_events","type":"Organization","site_admin":false},"repo":{"id":45871447,"name":"TaskListRails","full_name":"Ada-C4/TaskListRails","owner":{"login":"Ada-C4","id":14096372,"avatar_url":"https://avatars.githubusercontent.com/u/14096372?v=3","gravatar_id":"","url":"https://api.github.com/users/Ada-C4","html_url":"https://github.com/Ada-C4","followers_url":"https://api.github.com/users/Ada-C4/followers","following_url":"https://api.github.com/users/Ada-C4/following{/other_user}","gists_url":"https://api.github.com/users/Ada-C4/gists{/gist_id}","starred_url":"https://api.github.com/users/Ada-C4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ada-C4/subscriptions","organizations_url":"https://api.github.com/users/Ada-C4/orgs","repos_url":"https://api.github.com/users/Ada-C4/repos","events_url":"https://api.github.com/users/Ada-C4/events{/privacy}","received_events_url":"https://api.github.com/users/Ada-C4/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Ada-C4/TaskListRails","description":"","fork":true,"url":"https://api.github.com/repos/Ada-C4/TaskListRails","forks_url":"https://api.github.com/repos/Ada-C4/TaskListRails/forks","keys_url":"https://api.github.com/repos/Ada-C4/TaskListRails/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Ada-C4/TaskListRails/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Ada-C4/TaskListRails/teams","hooks_url":"https://api.github.com/repos/Ada-C4/TaskListRails/hooks","issue_events_url":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/events{/number}","events_url":"https://api.github.com/repos/Ada-C4/TaskListRails/events","assignees_url":"https://api.github.com/repos/Ada-C4/TaskListRails/assignees{/user}","branches_url":"https://api.github.com/repos/Ada-C4/TaskListRails/branches{/branch}","tags_url":"https://api.github.com/repos/Ada-C4/TaskListRails/tags","blobs_url":"https://api.github.com/repos/Ada-C4/TaskListRails/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Ada-C4/TaskListRails/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Ada-C4/TaskListRails/git/refs{/sha}","trees_url":"https://api.github.com/repos/Ada-C4/TaskListRails/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Ada-C4/TaskListRails/statuses/{sha}","languages_url":"https://api.github.com/repos/Ada-C4/TaskListRails/languages","stargazers_url":"https://api.github.com/repos/Ada-C4/TaskListRails/stargazers","contributors_url":"https://api.github.com/repos/Ada-C4/TaskListRails/contributors","subscribers_url":"https://api.github.com/repos/Ada-C4/TaskListRails/subscribers","subscription_url":"https://api.github.com/repos/Ada-C4/TaskListRails/subscription","commits_url":"https://api.github.com/repos/Ada-C4/TaskListRails/commits{/sha}","git_commits_url":"https://api.github.com/repos/Ada-C4/TaskListRails/git/commits{/sha}","comments_url":"https://api.github.com/repos/Ada-C4/TaskListRails/comments{/number}","issue_comment_url":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/comments{/number}","contents_url":"https://api.github.com/repos/Ada-C4/TaskListRails/contents/{+path}","compare_url":"https://api.github.com/repos/Ada-C4/TaskListRails/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Ada-C4/TaskListRails/merges","archive_url":"https://api.github.com/repos/Ada-C4/TaskListRails/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Ada-C4/TaskListRails/downloads","issues_url":"https://api.github.com/repos/Ada-C4/TaskListRails/issues{/number}","pulls_url":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls{/number}","milestones_url":"https://api.github.com/repos/Ada-C4/TaskListRails/milestones{/number}","notifications_url":"https://api.github.com/repos/Ada-C4/TaskListRails/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Ada-C4/TaskListRails/labels{/name}","releases_url":"https://api.github.com/repos/Ada-C4/TaskListRails/releases{/id}","created_at":"2015-11-09T22:36:26Z","updated_at":"2015-11-09T22:36:27Z","pushed_at":"2015-11-17T16:56:34Z","git_url":"git://github.com/Ada-C4/TaskListRails.git","ssh_url":"git@github.com:Ada-C4/TaskListRails.git","clone_url":"https://github.com/Ada-C4/TaskListRails.git","svn_url":"https://github.com/Ada-C4/TaskListRails","homepage":null,"size":21700,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":22,"mirror_url":null,"open_issues_count":22,"forks":22,"open_issues":22,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90"},"html":{"href":"https://github.com/Ada-C4/TaskListRails/pull/90"},"issue":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/90"},"comments":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/issues/90/comments"},"review_comments":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90/comments"},"review_comment":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/pulls/90/commits"},"statuses":{"href":"https://api.github.com/repos/Ada-C4/TaskListRails/statuses/f0b3649ba36f2220c88dd735e94baf2057591e55"}}}}