{"action":"created","comment":{"url":"https://api.github.com/repos/devscola/huertask-app/pulls/comments/92948612","pull_request_review_id":13470421,"id":92948612,"diff_hunk":"@@ -171,18 +171,15 @@ def filter(params, missing = true)\n           route_param :category_id do\n             post \"/\" do\n               person = Person.find_by_id(params[:id])\n-              person.dislike_categories.delete_if { |cat| p cat.id; cat.id == params[:category_id].to_i }\n-              if person.save\n+              if person.addFavoriteCategory(params[:category_id])\n                 present person, with: Entities::Person\n               else\n                 error_400(person)\n               end\n             end\n             delete \"/\" do\n               person = Person.find_by_id(params[:id])\n-              category = Category.find_by_id(params[:category_id])\n-              person.dislike_categories << category\n-              if person.save\n+              if person.removeFavoriteCategory(params[:category_id])","path":"api/main.rb","position":53,"original_position":17,"commit_id":"d6a3daf1fdf911edb1b5f6e02794933fe4296144","original_commit_id":"39986f1148ac1af84908a15a5ec9fae4d52926ae","user":{"login":"esebastian","id":577074,"avatar_url":"https://avatars.githubusercontent.com/u/577074?v=3","gravatar_id":"","url":"https://api.github.com/users/esebastian","html_url":"https://github.com/esebastian","followers_url":"https://api.github.com/users/esebastian/followers","following_url":"https://api.github.com/users/esebastian/following{/other_user}","gists_url":"https://api.github.com/users/esebastian/gists{/gist_id}","starred_url":"https://api.github.com/users/esebastian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/esebastian/subscriptions","organizations_url":"https://api.github.com/users/esebastian/orgs","repos_url":"https://api.github.com/users/esebastian/repos","events_url":"https://api.github.com/users/esebastian/events{/privacy}","received_events_url":"https://api.github.com/users/esebastian/received_events","type":"User","site_admin":false},"body":"the naming is not ruby-ish","created_at":"2016-12-18T19:24:44Z","updated_at":"2016-12-18T19:24:44Z","html_url":"https://github.com/devscola/huertask-app/pull/77#discussion_r92948612","pull_request_url":"https://api.github.com/repos/devscola/huertask-app/pulls/77","_links":{"self":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/comments/92948612"},"html":{"href":"https://github.com/devscola/huertask-app/pull/77#discussion_r92948612"},"pull_request":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/77"}}},"pull_request":{"url":"https://api.github.com/repos/devscola/huertask-app/pulls/77","id":98483354,"html_url":"https://github.com/devscola/huertask-app/pull/77","diff_url":"https://github.com/devscola/huertask-app/pull/77.diff","patch_url":"https://github.com/devscola/huertask-app/pull/77.patch","issue_url":"https://api.github.com/repos/devscola/huertask-app/issues/77","number":77,"state":"open","locked":false,"title":"Filter tasks by user & create category","user":{"login":"Pardiez","id":2436316,"avatar_url":"https://avatars.githubusercontent.com/u/2436316?v=3","gravatar_id":"","url":"https://api.github.com/users/Pardiez","html_url":"https://github.com/Pardiez","followers_url":"https://api.github.com/users/Pardiez/followers","following_url":"https://api.github.com/users/Pardiez/following{/other_user}","gists_url":"https://api.github.com/users/Pardiez/gists{/gist_id}","starred_url":"https://api.github.com/users/Pardiez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Pardiez/subscriptions","organizations_url":"https://api.github.com/users/Pardiez/orgs","repos_url":"https://api.github.com/users/Pardiez/repos","events_url":"https://api.github.com/users/Pardiez/events{/privacy}","received_events_url":"https://api.github.com/users/Pardiez/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-12-18T18:49:40Z","updated_at":"2016-12-18T19:24:44Z","closed_at":null,"merged_at":null,"merge_commit_sha":"a727448a19966ae04e2fca7b8b8a854d6dc54c24","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/devscola/huertask-app/pulls/77/commits","review_comments_url":"https://api.github.com/repos/devscola/huertask-app/pulls/77/comments","review_comment_url":"https://api.github.com/repos/devscola/huertask-app/pulls/comments{/number}","comments_url":"https://api.github.com/repos/devscola/huertask-app/issues/77/comments","statuses_url":"https://api.github.com/repos/devscola/huertask-app/statuses/d6a3daf1fdf911edb1b5f6e02794933fe4296144","head":{"label":"devscola:ft_fav_categories","ref":"ft_fav_categories","sha":"d6a3daf1fdf911edb1b5f6e02794933fe4296144","user":{"login":"devscola","id":6756717,"avatar_url":"https://avatars.githubusercontent.com/u/6756717?v=3","gravatar_id":"","url":"https://api.github.com/users/devscola","html_url":"https://github.com/devscola","followers_url":"https://api.github.com/users/devscola/followers","following_url":"https://api.github.com/users/devscola/following{/other_user}","gists_url":"https://api.github.com/users/devscola/gists{/gist_id}","starred_url":"https://api.github.com/users/devscola/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/devscola/subscriptions","organizations_url":"https://api.github.com/users/devscola/orgs","repos_url":"https://api.github.com/users/devscola/repos","events_url":"https://api.github.com/users/devscola/events{/privacy}","received_events_url":"https://api.github.com/users/devscola/received_events","type":"Organization","site_admin":false},"repo":{"id":72527559,"name":"huertask-app","full_name":"devscola/huertask-app","owner":{"login":"devscola","id":6756717,"avatar_url":"https://avatars.githubusercontent.com/u/6756717?v=3","gravatar_id":"","url":"https://api.github.com/users/devscola","html_url":"https://github.com/devscola","followers_url":"https://api.github.com/users/devscola/followers","following_url":"https://api.github.com/users/devscola/following{/other_user}","gists_url":"https://api.github.com/users/devscola/gists{/gist_id}","starred_url":"https://api.github.com/users/devscola/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/devscola/subscriptions","organizations_url":"https://api.github.com/users/devscola/orgs","repos_url":"https://api.github.com/users/devscola/repos","events_url":"https://api.github.com/users/devscola/events{/privacy}","received_events_url":"https://api.github.com/users/devscola/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/devscola/huertask-app","description":"mobile application for community garden management","fork":false,"url":"https://api.github.com/repos/devscola/huertask-app","forks_url":"https://api.github.com/repos/devscola/huertask-app/forks","keys_url":"https://api.github.com/repos/devscola/huertask-app/keys{/key_id}","collaborators_url":"https://api.github.com/repos/devscola/huertask-app/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/devscola/huertask-app/teams","hooks_url":"https://api.github.com/repos/devscola/huertask-app/hooks","issue_events_url":"https://api.github.com/repos/devscola/huertask-app/issues/events{/number}","events_url":"https://api.github.com/repos/devscola/huertask-app/events","assignees_url":"https://api.github.com/repos/devscola/huertask-app/assignees{/user}","branches_url":"https://api.github.com/repos/devscola/huertask-app/branches{/branch}","tags_url":"https://api.github.com/repos/devscola/huertask-app/tags","blobs_url":"https://api.github.com/repos/devscola/huertask-app/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/devscola/huertask-app/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/devscola/huertask-app/git/refs{/sha}","trees_url":"https://api.github.com/repos/devscola/huertask-app/git/trees{/sha}","statuses_url":"https://api.github.com/repos/devscola/huertask-app/statuses/{sha}","languages_url":"https://api.github.com/repos/devscola/huertask-app/languages","stargazers_url":"https://api.github.com/repos/devscola/huertask-app/stargazers","contributors_url":"https://api.github.com/repos/devscola/huertask-app/contributors","subscribers_url":"https://api.github.com/repos/devscola/huertask-app/subscribers","subscription_url":"https://api.github.com/repos/devscola/huertask-app/subscription","commits_url":"https://api.github.com/repos/devscola/huertask-app/commits{/sha}","git_commits_url":"https://api.github.com/repos/devscola/huertask-app/git/commits{/sha}","comments_url":"https://api.github.com/repos/devscola/huertask-app/comments{/number}","issue_comment_url":"https://api.github.com/repos/devscola/huertask-app/issues/comments{/number}","contents_url":"https://api.github.com/repos/devscola/huertask-app/contents/{+path}","compare_url":"https://api.github.com/repos/devscola/huertask-app/compare/{base}...{head}","merges_url":"https://api.github.com/repos/devscola/huertask-app/merges","archive_url":"https://api.github.com/repos/devscola/huertask-app/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/devscola/huertask-app/downloads","issues_url":"https://api.github.com/repos/devscola/huertask-app/issues{/number}","pulls_url":"https://api.github.com/repos/devscola/huertask-app/pulls{/number}","milestones_url":"https://api.github.com/repos/devscola/huertask-app/milestones{/number}","notifications_url":"https://api.github.com/repos/devscola/huertask-app/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/devscola/huertask-app/labels{/name}","releases_url":"https://api.github.com/repos/devscola/huertask-app/releases{/id}","deployments_url":"https://api.github.com/repos/devscola/huertask-app/deployments","created_at":"2016-11-01T10:48:43Z","updated_at":"2016-11-16T15:54:12Z","pushed_at":"2016-12-18T18:49:40Z","git_url":"git://github.com/devscola/huertask-app.git","ssh_url":"git@github.com:devscola/huertask-app.git","clone_url":"https://github.com/devscola/huertask-app.git","svn_url":"https://github.com/devscola/huertask-app","homepage":null,"size":1237,"stargazers_count":1,"watchers_count":1,"language":"TypeScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":2,"forks":0,"open_issues":2,"watchers":1,"default_branch":"develop"}},"base":{"label":"devscola:develop","ref":"develop","sha":"b4702b0ecdec8a42435e974deabb11289d84dc3e","user":{"login":"devscola","id":6756717,"avatar_url":"https://avatars.githubusercontent.com/u/6756717?v=3","gravatar_id":"","url":"https://api.github.com/users/devscola","html_url":"https://github.com/devscola","followers_url":"https://api.github.com/users/devscola/followers","following_url":"https://api.github.com/users/devscola/following{/other_user}","gists_url":"https://api.github.com/users/devscola/gists{/gist_id}","starred_url":"https://api.github.com/users/devscola/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/devscola/subscriptions","organizations_url":"https://api.github.com/users/devscola/orgs","repos_url":"https://api.github.com/users/devscola/repos","events_url":"https://api.github.com/users/devscola/events{/privacy}","received_events_url":"https://api.github.com/users/devscola/received_events","type":"Organization","site_admin":false},"repo":{"id":72527559,"name":"huertask-app","full_name":"devscola/huertask-app","owner":{"login":"devscola","id":6756717,"avatar_url":"https://avatars.githubusercontent.com/u/6756717?v=3","gravatar_id":"","url":"https://api.github.com/users/devscola","html_url":"https://github.com/devscola","followers_url":"https://api.github.com/users/devscola/followers","following_url":"https://api.github.com/users/devscola/following{/other_user}","gists_url":"https://api.github.com/users/devscola/gists{/gist_id}","starred_url":"https://api.github.com/users/devscola/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/devscola/subscriptions","organizations_url":"https://api.github.com/users/devscola/orgs","repos_url":"https://api.github.com/users/devscola/repos","events_url":"https://api.github.com/users/devscola/events{/privacy}","received_events_url":"https://api.github.com/users/devscola/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/devscola/huertask-app","description":"mobile application for community garden management","fork":false,"url":"https://api.github.com/repos/devscola/huertask-app","forks_url":"https://api.github.com/repos/devscola/huertask-app/forks","keys_url":"https://api.github.com/repos/devscola/huertask-app/keys{/key_id}","collaborators_url":"https://api.github.com/repos/devscola/huertask-app/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/devscola/huertask-app/teams","hooks_url":"https://api.github.com/repos/devscola/huertask-app/hooks","issue_events_url":"https://api.github.com/repos/devscola/huertask-app/issues/events{/number}","events_url":"https://api.github.com/repos/devscola/huertask-app/events","assignees_url":"https://api.github.com/repos/devscola/huertask-app/assignees{/user}","branches_url":"https://api.github.com/repos/devscola/huertask-app/branches{/branch}","tags_url":"https://api.github.com/repos/devscola/huertask-app/tags","blobs_url":"https://api.github.com/repos/devscola/huertask-app/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/devscola/huertask-app/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/devscola/huertask-app/git/refs{/sha}","trees_url":"https://api.github.com/repos/devscola/huertask-app/git/trees{/sha}","statuses_url":"https://api.github.com/repos/devscola/huertask-app/statuses/{sha}","languages_url":"https://api.github.com/repos/devscola/huertask-app/languages","stargazers_url":"https://api.github.com/repos/devscola/huertask-app/stargazers","contributors_url":"https://api.github.com/repos/devscola/huertask-app/contributors","subscribers_url":"https://api.github.com/repos/devscola/huertask-app/subscribers","subscription_url":"https://api.github.com/repos/devscola/huertask-app/subscription","commits_url":"https://api.github.com/repos/devscola/huertask-app/commits{/sha}","git_commits_url":"https://api.github.com/repos/devscola/huertask-app/git/commits{/sha}","comments_url":"https://api.github.com/repos/devscola/huertask-app/comments{/number}","issue_comment_url":"https://api.github.com/repos/devscola/huertask-app/issues/comments{/number}","contents_url":"https://api.github.com/repos/devscola/huertask-app/contents/{+path}","compare_url":"https://api.github.com/repos/devscola/huertask-app/compare/{base}...{head}","merges_url":"https://api.github.com/repos/devscola/huertask-app/merges","archive_url":"https://api.github.com/repos/devscola/huertask-app/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/devscola/huertask-app/downloads","issues_url":"https://api.github.com/repos/devscola/huertask-app/issues{/number}","pulls_url":"https://api.github.com/repos/devscola/huertask-app/pulls{/number}","milestones_url":"https://api.github.com/repos/devscola/huertask-app/milestones{/number}","notifications_url":"https://api.github.com/repos/devscola/huertask-app/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/devscola/huertask-app/labels{/name}","releases_url":"https://api.github.com/repos/devscola/huertask-app/releases{/id}","deployments_url":"https://api.github.com/repos/devscola/huertask-app/deployments","created_at":"2016-11-01T10:48:43Z","updated_at":"2016-11-16T15:54:12Z","pushed_at":"2016-12-18T18:49:40Z","git_url":"git://github.com/devscola/huertask-app.git","ssh_url":"git@github.com:devscola/huertask-app.git","clone_url":"https://github.com/devscola/huertask-app.git","svn_url":"https://github.com/devscola/huertask-app","homepage":null,"size":1237,"stargazers_count":1,"watchers_count":1,"language":"TypeScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":2,"forks":0,"open_issues":2,"watchers":1,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/77"},"html":{"href":"https://github.com/devscola/huertask-app/pull/77"},"issue":{"href":"https://api.github.com/repos/devscola/huertask-app/issues/77"},"comments":{"href":"https://api.github.com/repos/devscola/huertask-app/issues/77/comments"},"review_comments":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/77/comments"},"review_comment":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/devscola/huertask-app/pulls/77/commits"},"statuses":{"href":"https://api.github.com/repos/devscola/huertask-app/statuses/d6a3daf1fdf911edb1b5f6e02794933fe4296144"}}}}
{"actor":{"display_login":"esebastian"}}