{"action":"created","comment":{"url":"https://api.github.com/repos/akintner/scoopful/pulls/comments/94891391","pull_request_review_id":15433205,"id":94891391,"diff_hunk":"@@ -16,6 +18,13 @@ def index\n     @items = @cart.items\n   end\n \n+  def update\n+    @cart.update_item(cart_params)","path":"app/controllers/carts_controller.rb","position":14,"original_position":14,"commit_id":"634ca76ab6fddb3426fa20c15c44060681f09e1e","original_commit_id":"6d173aa9c5d9d20df285e348fa1f13f155a015c1","user":{"login":"tmikeschu","id":19894032,"avatar_url":"https://avatars.githubusercontent.com/u/19894032?v=3","gravatar_id":"","url":"https://api.github.com/users/tmikeschu","html_url":"https://github.com/tmikeschu","followers_url":"https://api.github.com/users/tmikeschu/followers","following_url":"https://api.github.com/users/tmikeschu/following{/other_user}","gists_url":"https://api.github.com/users/tmikeschu/gists{/gist_id}","starred_url":"https://api.github.com/users/tmikeschu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmikeschu/subscriptions","organizations_url":"https://api.github.com/users/tmikeschu/orgs","repos_url":"https://api.github.com/users/tmikeschu/repos","events_url":"https://api.github.com/users/tmikeschu/events{/privacy}","received_events_url":"https://api.github.com/users/tmikeschu/received_events","type":"User","site_admin":false},"body":"@kheppenstall I did the permit just to simplify the passed hash. I'm fine with just passing params, as I see it might be an unnecessary use of strong params.","created_at":"2017-01-06T02:24:01Z","updated_at":"2017-01-06T02:24:01Z","html_url":"https://github.com/akintner/scoopful/pull/85#discussion_r94891391","pull_request_url":"https://api.github.com/repos/akintner/scoopful/pulls/85","_links":{"self":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/comments/94891391"},"html":{"href":"https://github.com/akintner/scoopful/pull/85#discussion_r94891391"},"pull_request":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/85"}}},"pull_request":{"url":"https://api.github.com/repos/akintner/scoopful/pulls/85","id":100393772,"html_url":"https://github.com/akintner/scoopful/pull/85","diff_url":"https://github.com/akintner/scoopful/pull/85.diff","patch_url":"https://github.com/akintner/scoopful/pull/85.patch","issue_url":"https://api.github.com/repos/akintner/scoopful/issues/85","number":85,"state":"open","locked":false,"title":"Tms update quantity #72","user":{"login":"tmikeschu","id":19894032,"avatar_url":"https://avatars.githubusercontent.com/u/19894032?v=3","gravatar_id":"","url":"https://api.github.com/users/tmikeschu","html_url":"https://github.com/tmikeschu","followers_url":"https://api.github.com/users/tmikeschu/followers","following_url":"https://api.github.com/users/tmikeschu/following{/other_user}","gists_url":"https://api.github.com/users/tmikeschu/gists{/gist_id}","starred_url":"https://api.github.com/users/tmikeschu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tmikeschu/subscriptions","organizations_url":"https://api.github.com/users/tmikeschu/orgs","repos_url":"https://api.github.com/users/tmikeschu/repos","events_url":"https://api.github.com/users/tmikeschu/events{/privacy}","received_events_url":"https://api.github.com/users/tmikeschu/received_events","type":"User","site_admin":false},"body":"- Add a form to the cart index to update item quantity. It acts as a reset, rather than an incrementer. The number field displays the current amount by default, say 4, and whatever the user clicks up or down to becomes set as the new value, say 2.\r\n\r\n- Add a subtotal method to cart to calculate the subtotal for a given item\r\n\r\nCloses #72 \r\n\r\n<img width=\"286\" alt=\"screen shot 2017-01-05 at 6 24 01 pm\" src=\"https://cloud.githubusercontent.com/assets/19894032/21704779/b10f5e3e-d378-11e6-8270-27d21c4136ca.png\">\r\n<img width=\"695\" alt=\"screen shot 2017-01-05 at 6 56 24 pm\" src=\"https://cloud.githubusercontent.com/assets/19894032/21704778/b10e8cfc-d378-11e6-97d2-8e3f7ae1da9d.png\">\r\n","created_at":"2017-01-06T01:04:15Z","updated_at":"2017-01-06T02:24:01Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f11d8c9a2f2d3be14eb1daec683f56ff375dfac8","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/akintner/scoopful/pulls/85/commits","review_comments_url":"https://api.github.com/repos/akintner/scoopful/pulls/85/comments","review_comment_url":"https://api.github.com/repos/akintner/scoopful/pulls/comments{/number}","comments_url":"https://api.github.com/repos/akintner/scoopful/issues/85/comments","statuses_url":"https://api.github.com/repos/akintner/scoopful/statuses/634ca76ab6fddb3426fa20c15c44060681f09e1e","head":{"label":"akintner:tms-update-quantity-#72","ref":"tms-update-quantity-#72","sha":"634ca76ab6fddb3426fa20c15c44060681f09e1e","user":{"login":"akintner","id":20585744,"avatar_url":"https://avatars.githubusercontent.com/u/20585744?v=3","gravatar_id":"","url":"https://api.github.com/users/akintner","html_url":"https://github.com/akintner","followers_url":"https://api.github.com/users/akintner/followers","following_url":"https://api.github.com/users/akintner/following{/other_user}","gists_url":"https://api.github.com/users/akintner/gists{/gist_id}","starred_url":"https://api.github.com/users/akintner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akintner/subscriptions","organizations_url":"https://api.github.com/users/akintner/orgs","repos_url":"https://api.github.com/users/akintner/repos","events_url":"https://api.github.com/users/akintner/events{/privacy}","received_events_url":"https://api.github.com/users/akintner/received_events","type":"User","site_admin":false},"repo":{"id":77953639,"name":"scoopful","full_name":"akintner/scoopful","owner":{"login":"akintner","id":20585744,"avatar_url":"https://avatars.githubusercontent.com/u/20585744?v=3","gravatar_id":"","url":"https://api.github.com/users/akintner","html_url":"https://github.com/akintner","followers_url":"https://api.github.com/users/akintner/followers","following_url":"https://api.github.com/users/akintner/following{/other_user}","gists_url":"https://api.github.com/users/akintner/gists{/gist_id}","starred_url":"https://api.github.com/users/akintner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akintner/subscriptions","organizations_url":"https://api.github.com/users/akintner/orgs","repos_url":"https://api.github.com/users/akintner/repos","events_url":"https://api.github.com/users/akintner/events{/privacy}","received_events_url":"https://api.github.com/users/akintner/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/akintner/scoopful","description":"online cooking store with recipes for which we'll ship you as many of the ingredients as you need","fork":false,"url":"https://api.github.com/repos/akintner/scoopful","forks_url":"https://api.github.com/repos/akintner/scoopful/forks","keys_url":"https://api.github.com/repos/akintner/scoopful/keys{/key_id}","collaborators_url":"https://api.github.com/repos/akintner/scoopful/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/akintner/scoopful/teams","hooks_url":"https://api.github.com/repos/akintner/scoopful/hooks","issue_events_url":"https://api.github.com/repos/akintner/scoopful/issues/events{/number}","events_url":"https://api.github.com/repos/akintner/scoopful/events","assignees_url":"https://api.github.com/repos/akintner/scoopful/assignees{/user}","branches_url":"https://api.github.com/repos/akintner/scoopful/branches{/branch}","tags_url":"https://api.github.com/repos/akintner/scoopful/tags","blobs_url":"https://api.github.com/repos/akintner/scoopful/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/akintner/scoopful/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/akintner/scoopful/git/refs{/sha}","trees_url":"https://api.github.com/repos/akintner/scoopful/git/trees{/sha}","statuses_url":"https://api.github.com/repos/akintner/scoopful/statuses/{sha}","languages_url":"https://api.github.com/repos/akintner/scoopful/languages","stargazers_url":"https://api.github.com/repos/akintner/scoopful/stargazers","contributors_url":"https://api.github.com/repos/akintner/scoopful/contributors","subscribers_url":"https://api.github.com/repos/akintner/scoopful/subscribers","subscription_url":"https://api.github.com/repos/akintner/scoopful/subscription","commits_url":"https://api.github.com/repos/akintner/scoopful/commits{/sha}","git_commits_url":"https://api.github.com/repos/akintner/scoopful/git/commits{/sha}","comments_url":"https://api.github.com/repos/akintner/scoopful/comments{/number}","issue_comment_url":"https://api.github.com/repos/akintner/scoopful/issues/comments{/number}","contents_url":"https://api.github.com/repos/akintner/scoopful/contents/{+path}","compare_url":"https://api.github.com/repos/akintner/scoopful/compare/{base}...{head}","merges_url":"https://api.github.com/repos/akintner/scoopful/merges","archive_url":"https://api.github.com/repos/akintner/scoopful/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/akintner/scoopful/downloads","issues_url":"https://api.github.com/repos/akintner/scoopful/issues{/number}","pulls_url":"https://api.github.com/repos/akintner/scoopful/pulls{/number}","milestones_url":"https://api.github.com/repos/akintner/scoopful/milestones{/number}","notifications_url":"https://api.github.com/repos/akintner/scoopful/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/akintner/scoopful/labels{/name}","releases_url":"https://api.github.com/repos/akintner/scoopful/releases{/id}","deployments_url":"https://api.github.com/repos/akintner/scoopful/deployments","created_at":"2017-01-03T21:10:13Z","updated_at":"2017-01-03T23:01:33Z","pushed_at":"2017-01-06T02:21:07Z","git_url":"git://github.com/akintner/scoopful.git","ssh_url":"git@github.com:akintner/scoopful.git","clone_url":"https://github.com/akintner/scoopful.git","svn_url":"https://github.com/akintner/scoopful","homepage":null,"size":11303,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":14,"forks":0,"open_issues":14,"watchers":0,"default_branch":"master"}},"base":{"label":"akintner:master","ref":"master","sha":"ba4099cf71510d44c8adf27b6810fe694b6bc2da","user":{"login":"akintner","id":20585744,"avatar_url":"https://avatars.githubusercontent.com/u/20585744?v=3","gravatar_id":"","url":"https://api.github.com/users/akintner","html_url":"https://github.com/akintner","followers_url":"https://api.github.com/users/akintner/followers","following_url":"https://api.github.com/users/akintner/following{/other_user}","gists_url":"https://api.github.com/users/akintner/gists{/gist_id}","starred_url":"https://api.github.com/users/akintner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akintner/subscriptions","organizations_url":"https://api.github.com/users/akintner/orgs","repos_url":"https://api.github.com/users/akintner/repos","events_url":"https://api.github.com/users/akintner/events{/privacy}","received_events_url":"https://api.github.com/users/akintner/received_events","type":"User","site_admin":false},"repo":{"id":77953639,"name":"scoopful","full_name":"akintner/scoopful","owner":{"login":"akintner","id":20585744,"avatar_url":"https://avatars.githubusercontent.com/u/20585744?v=3","gravatar_id":"","url":"https://api.github.com/users/akintner","html_url":"https://github.com/akintner","followers_url":"https://api.github.com/users/akintner/followers","following_url":"https://api.github.com/users/akintner/following{/other_user}","gists_url":"https://api.github.com/users/akintner/gists{/gist_id}","starred_url":"https://api.github.com/users/akintner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akintner/subscriptions","organizations_url":"https://api.github.com/users/akintner/orgs","repos_url":"https://api.github.com/users/akintner/repos","events_url":"https://api.github.com/users/akintner/events{/privacy}","received_events_url":"https://api.github.com/users/akintner/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/akintner/scoopful","description":"online cooking store with recipes for which we'll ship you as many of the ingredients as you need","fork":false,"url":"https://api.github.com/repos/akintner/scoopful","forks_url":"https://api.github.com/repos/akintner/scoopful/forks","keys_url":"https://api.github.com/repos/akintner/scoopful/keys{/key_id}","collaborators_url":"https://api.github.com/repos/akintner/scoopful/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/akintner/scoopful/teams","hooks_url":"https://api.github.com/repos/akintner/scoopful/hooks","issue_events_url":"https://api.github.com/repos/akintner/scoopful/issues/events{/number}","events_url":"https://api.github.com/repos/akintner/scoopful/events","assignees_url":"https://api.github.com/repos/akintner/scoopful/assignees{/user}","branches_url":"https://api.github.com/repos/akintner/scoopful/branches{/branch}","tags_url":"https://api.github.com/repos/akintner/scoopful/tags","blobs_url":"https://api.github.com/repos/akintner/scoopful/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/akintner/scoopful/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/akintner/scoopful/git/refs{/sha}","trees_url":"https://api.github.com/repos/akintner/scoopful/git/trees{/sha}","statuses_url":"https://api.github.com/repos/akintner/scoopful/statuses/{sha}","languages_url":"https://api.github.com/repos/akintner/scoopful/languages","stargazers_url":"https://api.github.com/repos/akintner/scoopful/stargazers","contributors_url":"https://api.github.com/repos/akintner/scoopful/contributors","subscribers_url":"https://api.github.com/repos/akintner/scoopful/subscribers","subscription_url":"https://api.github.com/repos/akintner/scoopful/subscription","commits_url":"https://api.github.com/repos/akintner/scoopful/commits{/sha}","git_commits_url":"https://api.github.com/repos/akintner/scoopful/git/commits{/sha}","comments_url":"https://api.github.com/repos/akintner/scoopful/comments{/number}","issue_comment_url":"https://api.github.com/repos/akintner/scoopful/issues/comments{/number}","contents_url":"https://api.github.com/repos/akintner/scoopful/contents/{+path}","compare_url":"https://api.github.com/repos/akintner/scoopful/compare/{base}...{head}","merges_url":"https://api.github.com/repos/akintner/scoopful/merges","archive_url":"https://api.github.com/repos/akintner/scoopful/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/akintner/scoopful/downloads","issues_url":"https://api.github.com/repos/akintner/scoopful/issues{/number}","pulls_url":"https://api.github.com/repos/akintner/scoopful/pulls{/number}","milestones_url":"https://api.github.com/repos/akintner/scoopful/milestones{/number}","notifications_url":"https://api.github.com/repos/akintner/scoopful/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/akintner/scoopful/labels{/name}","releases_url":"https://api.github.com/repos/akintner/scoopful/releases{/id}","deployments_url":"https://api.github.com/repos/akintner/scoopful/deployments","created_at":"2017-01-03T21:10:13Z","updated_at":"2017-01-03T23:01:33Z","pushed_at":"2017-01-06T02:21:07Z","git_url":"git://github.com/akintner/scoopful.git","ssh_url":"git@github.com:akintner/scoopful.git","clone_url":"https://github.com/akintner/scoopful.git","svn_url":"https://github.com/akintner/scoopful","homepage":null,"size":11303,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":14,"forks":0,"open_issues":14,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/85"},"html":{"href":"https://github.com/akintner/scoopful/pull/85"},"issue":{"href":"https://api.github.com/repos/akintner/scoopful/issues/85"},"comments":{"href":"https://api.github.com/repos/akintner/scoopful/issues/85/comments"},"review_comments":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/85/comments"},"review_comment":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/akintner/scoopful/pulls/85/commits"},"statuses":{"href":"https://api.github.com/repos/akintner/scoopful/statuses/634ca76ab6fddb3426fa20c15c44060681f09e1e"}}}}
{"actor":{"display_login":"tmikeschu"}}