{"action":"created","comment":{"url":"https://api.github.com/repos/sixthedge/cellar/pulls/comments/115908073","pull_request_review_id":37485602,"id":115908073,"diff_hunk":"@@ -1,6 +1,10 @@\n module Thinkspace\n   module ReadinessAssurance\n     class Assessment < ActiveRecord::Base\n+      ## Can get the after_save working via callbacks (http://stackoverflow.com/questions/1342761/how-to-skip-activerecord-callbacks?noredirect=1&lq=1)","path":"src/thinkspace/api/thinkspace-readiness-assurance/app/models/thinkspace/readiness_assurance/assessment.rb","position":4,"original_position":4,"commit_id":"94f77df6aab88a1b99eacceb1edae583d9ea397b","original_commit_id":"94f77df6aab88a1b99eacceb1edae583d9ea397b","user":{"login":"jamesfid","id":215912,"avatar_url":"https://avatars0.githubusercontent.com/u/215912?v=3","gravatar_id":"","url":"https://api.github.com/users/jamesfid","html_url":"https://github.com/jamesfid","followers_url":"https://api.github.com/users/jamesfid/followers","following_url":"https://api.github.com/users/jamesfid/following{/other_user}","gists_url":"https://api.github.com/users/jamesfid/gists{/gist_id}","starred_url":"https://api.github.com/users/jamesfid/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamesfid/subscriptions","organizations_url":"https://api.github.com/users/jamesfid/orgs","repos_url":"https://api.github.com/users/jamesfid/repos","events_url":"https://api.github.com/users/jamesfid/events{/privacy}","received_events_url":"https://api.github.com/users/jamesfid/received_events","type":"User","site_admin":false},"body":"Look at: http://edgeguides.rubyonrails.org/active_record_callbacks.html#skipping-callbacks - it looks like we can use `update_column` to avoid the `after_save`.  There's a real use case where we end up modifying the JSON directly through a `rails c`.  Forgetting to update the tRAT would be easy.  I think we should try and hook up the `after_save` for now.","created_at":"2017-05-11T05:38:18Z","updated_at":"2017-05-11T06:02:12Z","html_url":"https://github.com/sixthedge/cellar/pull/18#discussion_r115908073","pull_request_url":"https://api.github.com/repos/sixthedge/cellar/pulls/18","_links":{"self":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/comments/115908073"},"html":{"href":"https://github.com/sixthedge/cellar/pull/18#discussion_r115908073"},"pull_request":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/18"}}},"pull_request":{"url":"https://api.github.com/repos/sixthedge/cellar/pulls/18","id":120004568,"html_url":"https://github.com/sixthedge/cellar/pull/18","diff_url":"https://github.com/sixthedge/cellar/pull/18.diff","patch_url":"https://github.com/sixthedge/cellar/pull/18.patch","issue_url":"https://api.github.com/repos/sixthedge/cellar/issues/18","number":18,"state":"open","locked":false,"title":"[RFR] Rat-builder merge into development","user":{"login":"DylanBell","id":2382826,"avatar_url":"https://avatars3.githubusercontent.com/u/2382826?v=3","gravatar_id":"","url":"https://api.github.com/users/DylanBell","html_url":"https://github.com/DylanBell","followers_url":"https://api.github.com/users/DylanBell/followers","following_url":"https://api.github.com/users/DylanBell/following{/other_user}","gists_url":"https://api.github.com/users/DylanBell/gists{/gist_id}","starred_url":"https://api.github.com/users/DylanBell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DylanBell/subscriptions","organizations_url":"https://api.github.com/users/DylanBell/orgs","repos_url":"https://api.github.com/users/DylanBell/repos","events_url":"https://api.github.com/users/DylanBell/events{/privacy}","received_events_url":"https://api.github.com/users/DylanBell/received_events","type":"User","site_admin":false},"body":"Adds rat-builder functionality for both client and api\r\n\r\nClient:\r\nCreates the new `thinkspace-builder-rat` routable engine, which handles the client-side code for the rat-builder.\r\n\r\nApi:\r\nModifies the Thinkspace::Casespace::Assignment and Thinkspace::ReadinessAssurance::Assessment models to support assessment syncing and handle rat creation/updating.","created_at":"2017-05-11T00:20:09Z","updated_at":"2017-05-11T06:02:13Z","closed_at":null,"merged_at":null,"merge_commit_sha":"79ae255b5b9ff21d5b4f64cb0347048bf0bbcdd6","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/sixthedge/cellar/pulls/18/commits","review_comments_url":"https://api.github.com/repos/sixthedge/cellar/pulls/18/comments","review_comment_url":"https://api.github.com/repos/sixthedge/cellar/pulls/comments{/number}","comments_url":"https://api.github.com/repos/sixthedge/cellar/issues/18/comments","statuses_url":"https://api.github.com/repos/sixthedge/cellar/statuses/94f77df6aab88a1b99eacceb1edae583d9ea397b","head":{"label":"sixthedge:feature/rat-builder","ref":"feature/rat-builder","sha":"94f77df6aab88a1b99eacceb1edae583d9ea397b","user":{"login":"sixthedge","id":11180832,"avatar_url":"https://avatars3.githubusercontent.com/u/11180832?v=3","gravatar_id":"","url":"https://api.github.com/users/sixthedge","html_url":"https://github.com/sixthedge","followers_url":"https://api.github.com/users/sixthedge/followers","following_url":"https://api.github.com/users/sixthedge/following{/other_user}","gists_url":"https://api.github.com/users/sixthedge/gists{/gist_id}","starred_url":"https://api.github.com/users/sixthedge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sixthedge/subscriptions","organizations_url":"https://api.github.com/users/sixthedge/orgs","repos_url":"https://api.github.com/users/sixthedge/repos","events_url":"https://api.github.com/users/sixthedge/events{/privacy}","received_events_url":"https://api.github.com/users/sixthedge/received_events","type":"Organization","site_admin":false},"repo":{"id":83590184,"name":"cellar","full_name":"sixthedge/cellar","owner":{"login":"sixthedge","id":11180832,"avatar_url":"https://avatars3.githubusercontent.com/u/11180832?v=3","gravatar_id":"","url":"https://api.github.com/users/sixthedge","html_url":"https://github.com/sixthedge","followers_url":"https://api.github.com/users/sixthedge/followers","following_url":"https://api.github.com/users/sixthedge/following{/other_user}","gists_url":"https://api.github.com/users/sixthedge/gists{/gist_id}","starred_url":"https://api.github.com/users/sixthedge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sixthedge/subscriptions","organizations_url":"https://api.github.com/users/sixthedge/orgs","repos_url":"https://api.github.com/users/sixthedge/repos","events_url":"https://api.github.com/users/sixthedge/events{/privacy}","received_events_url":"https://api.github.com/users/sixthedge/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/sixthedge/cellar","description":null,"fork":false,"url":"https://api.github.com/repos/sixthedge/cellar","forks_url":"https://api.github.com/repos/sixthedge/cellar/forks","keys_url":"https://api.github.com/repos/sixthedge/cellar/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sixthedge/cellar/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sixthedge/cellar/teams","hooks_url":"https://api.github.com/repos/sixthedge/cellar/hooks","issue_events_url":"https://api.github.com/repos/sixthedge/cellar/issues/events{/number}","events_url":"https://api.github.com/repos/sixthedge/cellar/events","assignees_url":"https://api.github.com/repos/sixthedge/cellar/assignees{/user}","branches_url":"https://api.github.com/repos/sixthedge/cellar/branches{/branch}","tags_url":"https://api.github.com/repos/sixthedge/cellar/tags","blobs_url":"https://api.github.com/repos/sixthedge/cellar/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sixthedge/cellar/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sixthedge/cellar/git/refs{/sha}","trees_url":"https://api.github.com/repos/sixthedge/cellar/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sixthedge/cellar/statuses/{sha}","languages_url":"https://api.github.com/repos/sixthedge/cellar/languages","stargazers_url":"https://api.github.com/repos/sixthedge/cellar/stargazers","contributors_url":"https://api.github.com/repos/sixthedge/cellar/contributors","subscribers_url":"https://api.github.com/repos/sixthedge/cellar/subscribers","subscription_url":"https://api.github.com/repos/sixthedge/cellar/subscription","commits_url":"https://api.github.com/repos/sixthedge/cellar/commits{/sha}","git_commits_url":"https://api.github.com/repos/sixthedge/cellar/git/commits{/sha}","comments_url":"https://api.github.com/repos/sixthedge/cellar/comments{/number}","issue_comment_url":"https://api.github.com/repos/sixthedge/cellar/issues/comments{/number}","contents_url":"https://api.github.com/repos/sixthedge/cellar/contents/{+path}","compare_url":"https://api.github.com/repos/sixthedge/cellar/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sixthedge/cellar/merges","archive_url":"https://api.github.com/repos/sixthedge/cellar/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sixthedge/cellar/downloads","issues_url":"https://api.github.com/repos/sixthedge/cellar/issues{/number}","pulls_url":"https://api.github.com/repos/sixthedge/cellar/pulls{/number}","milestones_url":"https://api.github.com/repos/sixthedge/cellar/milestones{/number}","notifications_url":"https://api.github.com/repos/sixthedge/cellar/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sixthedge/cellar/labels{/name}","releases_url":"https://api.github.com/repos/sixthedge/cellar/releases{/id}","deployments_url":"https://api.github.com/repos/sixthedge/cellar/deployments","created_at":"2017-03-01T19:01:31Z","updated_at":"2017-03-07T20:44:20Z","pushed_at":"2017-05-11T05:21:27Z","git_url":"git://github.com/sixthedge/cellar.git","ssh_url":"git@github.com:sixthedge/cellar.git","clone_url":"https://github.com/sixthedge/cellar.git","svn_url":"https://github.com/sixthedge/cellar","homepage":null,"size":6398,"stargazers_count":3,"watchers_count":3,"language":"Ruby","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":4,"forks":0,"open_issues":4,"watchers":3,"default_branch":"development"}},"base":{"label":"sixthedge:development","ref":"development","sha":"41f06cbf0cb334c6173f3ace9c77d7b6b7e21ad0","user":{"login":"sixthedge","id":11180832,"avatar_url":"https://avatars3.githubusercontent.com/u/11180832?v=3","gravatar_id":"","url":"https://api.github.com/users/sixthedge","html_url":"https://github.com/sixthedge","followers_url":"https://api.github.com/users/sixthedge/followers","following_url":"https://api.github.com/users/sixthedge/following{/other_user}","gists_url":"https://api.github.com/users/sixthedge/gists{/gist_id}","starred_url":"https://api.github.com/users/sixthedge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sixthedge/subscriptions","organizations_url":"https://api.github.com/users/sixthedge/orgs","repos_url":"https://api.github.com/users/sixthedge/repos","events_url":"https://api.github.com/users/sixthedge/events{/privacy}","received_events_url":"https://api.github.com/users/sixthedge/received_events","type":"Organization","site_admin":false},"repo":{"id":83590184,"name":"cellar","full_name":"sixthedge/cellar","owner":{"login":"sixthedge","id":11180832,"avatar_url":"https://avatars3.githubusercontent.com/u/11180832?v=3","gravatar_id":"","url":"https://api.github.com/users/sixthedge","html_url":"https://github.com/sixthedge","followers_url":"https://api.github.com/users/sixthedge/followers","following_url":"https://api.github.com/users/sixthedge/following{/other_user}","gists_url":"https://api.github.com/users/sixthedge/gists{/gist_id}","starred_url":"https://api.github.com/users/sixthedge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sixthedge/subscriptions","organizations_url":"https://api.github.com/users/sixthedge/orgs","repos_url":"https://api.github.com/users/sixthedge/repos","events_url":"https://api.github.com/users/sixthedge/events{/privacy}","received_events_url":"https://api.github.com/users/sixthedge/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/sixthedge/cellar","description":null,"fork":false,"url":"https://api.github.com/repos/sixthedge/cellar","forks_url":"https://api.github.com/repos/sixthedge/cellar/forks","keys_url":"https://api.github.com/repos/sixthedge/cellar/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sixthedge/cellar/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sixthedge/cellar/teams","hooks_url":"https://api.github.com/repos/sixthedge/cellar/hooks","issue_events_url":"https://api.github.com/repos/sixthedge/cellar/issues/events{/number}","events_url":"https://api.github.com/repos/sixthedge/cellar/events","assignees_url":"https://api.github.com/repos/sixthedge/cellar/assignees{/user}","branches_url":"https://api.github.com/repos/sixthedge/cellar/branches{/branch}","tags_url":"https://api.github.com/repos/sixthedge/cellar/tags","blobs_url":"https://api.github.com/repos/sixthedge/cellar/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sixthedge/cellar/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sixthedge/cellar/git/refs{/sha}","trees_url":"https://api.github.com/repos/sixthedge/cellar/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sixthedge/cellar/statuses/{sha}","languages_url":"https://api.github.com/repos/sixthedge/cellar/languages","stargazers_url":"https://api.github.com/repos/sixthedge/cellar/stargazers","contributors_url":"https://api.github.com/repos/sixthedge/cellar/contributors","subscribers_url":"https://api.github.com/repos/sixthedge/cellar/subscribers","subscription_url":"https://api.github.com/repos/sixthedge/cellar/subscription","commits_url":"https://api.github.com/repos/sixthedge/cellar/commits{/sha}","git_commits_url":"https://api.github.com/repos/sixthedge/cellar/git/commits{/sha}","comments_url":"https://api.github.com/repos/sixthedge/cellar/comments{/number}","issue_comment_url":"https://api.github.com/repos/sixthedge/cellar/issues/comments{/number}","contents_url":"https://api.github.com/repos/sixthedge/cellar/contents/{+path}","compare_url":"https://api.github.com/repos/sixthedge/cellar/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sixthedge/cellar/merges","archive_url":"https://api.github.com/repos/sixthedge/cellar/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sixthedge/cellar/downloads","issues_url":"https://api.github.com/repos/sixthedge/cellar/issues{/number}","pulls_url":"https://api.github.com/repos/sixthedge/cellar/pulls{/number}","milestones_url":"https://api.github.com/repos/sixthedge/cellar/milestones{/number}","notifications_url":"https://api.github.com/repos/sixthedge/cellar/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sixthedge/cellar/labels{/name}","releases_url":"https://api.github.com/repos/sixthedge/cellar/releases{/id}","deployments_url":"https://api.github.com/repos/sixthedge/cellar/deployments","created_at":"2017-03-01T19:01:31Z","updated_at":"2017-03-07T20:44:20Z","pushed_at":"2017-05-11T05:21:27Z","git_url":"git://github.com/sixthedge/cellar.git","ssh_url":"git@github.com:sixthedge/cellar.git","clone_url":"https://github.com/sixthedge/cellar.git","svn_url":"https://github.com/sixthedge/cellar","homepage":null,"size":6398,"stargazers_count":3,"watchers_count":3,"language":"Ruby","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":4,"forks":0,"open_issues":4,"watchers":3,"default_branch":"development"}},"_links":{"self":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/18"},"html":{"href":"https://github.com/sixthedge/cellar/pull/18"},"issue":{"href":"https://api.github.com/repos/sixthedge/cellar/issues/18"},"comments":{"href":"https://api.github.com/repos/sixthedge/cellar/issues/18/comments"},"review_comments":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/18/comments"},"review_comment":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/sixthedge/cellar/pulls/18/commits"},"statuses":{"href":"https://api.github.com/repos/sixthedge/cellar/statuses/94f77df6aab88a1b99eacceb1edae583d9ea397b"}}}}
{"actor":{"display_login":"jamesfid"}}