GitHub PullRequestReviewCommentEvent at 2014-01-29 20:44:32
Repository: wdi_4_rails_hw_activerecord_bookmarks
Actor: grantovich
Payload: {"comment":{"url":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pulls/comments/9283590","id":9283590,"diff_hunk":"@@ -0,0 +1,9 @@\n+<h1>Your current Bookmarks</h1>\n+<p><%= link_to 'New Bookmark', new_bookmark_path %></p>\n+<% @bookmarks.each do | bm | %>\n+\t<% if bm.favorite == true %>\n+<p><b><%= link_to bm.title, bm %></b></p>","path":"app/views/bookmarks/index.html.erb","position":5,"original_position":5,"commit_id":"0728454b432b4c47127a5681a34ec846b579b05d","original_commit_id":"0728454b432b4c47127a5681a34ec846b579b05d","user":{"login":"grantovich","id":394835,"avatar_url":"https://gravatar.com/avatar/b6c92201404bd9b616fbba5e783cb3d8?d=https%3A%2F%2Fidenticons.github.com%2F50bf48b85ed71ba0f25be359e2495785.png&r=x","gravatar_id":"b6c92201404bd9b616fbba5e783cb3d8","url":"https://api.github.com/users/grantovich","html_url":"https://github.com/grantovich","followers_url":"https://api.github.com/users/grantovich/followers","following_url":"https://api.github.com/users/grantovich/following{/other_user}","gists_url":"https://api.github.com/users/grantovich/gists{/gist_id}","starred_url":"https://api.github.com/users/grantovich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grantovich/subscriptions","organizations_url":"https://api.github.com/users/grantovich/orgs","repos_url":"https://api.github.com/users/grantovich/repos","events_url":"https://api.github.com/users/grantovich/events{/privacy}","received_events_url":"https://api.github.com/users/grantovich/received_events","type":"User","site_admin":false},"body":"This and line 7 should be indented two levels (four spaces). This puts them visually \"inside\" both the `each` loop and the `if` block, making it easier to see the logic that would cause them to be output.","created_at":"2014-01-29T20:44:32Z","updated_at":"2014-01-29T20:44:32Z","html_url":"https://github.com/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pull/12#discussion_r9283590","pull_request_url":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pulls/12","_links":{"self":{"href":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pulls/comments/9283590"},"html":{"href":"https://github.com/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pull/12#discussion_r9283590"},"pull_request":{"href":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_activerecord_bookmarks/pulls/12"}}}}