GitHub PullRequestReviewCommentEvent at 2014-04-11 04:47:48
Repository: roots-of-success
Actor: houndci
Payload: {"comment":{"url":"https://api.github.com/repos/calblueprint/roots-of-success/pulls/comments/11517981","id":11517981,"diff_hunk":"@@ -27,13 +27,18 @@ def show\n   def update\n     @module = LearningModule.find params[:id]\n \n-    @module.presentation_embed_code = params[:learning_module][:presentation_embed_code]\n-    @module.save\n+    @module.update_attributes module_attributes\n+    flash[:success] = 'Module updated!'\n \n     redirect_to @module\n   end\n \n   def destroy\n   end\n+\n+  private\n+    def module_attributes\n+      params.require(:learning_module).permit(:presentation_embed_code, :learning_module_file)","path":"app/controllers/learning_modules_controller.rb","position":17,"original_position":17,"commit_id":"857312614a780a5b469ebf40dfbb16b229dc16b9","original_commit_id":"857312614a780a5b469ebf40dfbb16b229dc16b9","user":{"login":"houndci","id":6697940,"avatar_url":"https://avatars.githubusercontent.com/u/6697940?","gravatar_id":"38d9cefd9971a05bf447f07c6cd332cc","url":"https://api.github.com/users/houndci","html_url":"https://github.com/houndci","followers_url":"https://api.github.com/users/houndci/followers","following_url":"https://api.github.com/users/houndci/following{/other_user}","gists_url":"https://api.github.com/users/houndci/gists{/gist_id}","starred_url":"https://api.github.com/users/houndci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/houndci/subscriptions","organizations_url":"https://api.github.com/users/houndci/orgs","repos_url":"https://api.github.com/users/houndci/repos","events_url":"https://api.github.com/users/houndci/events{/privacy}","received_events_url":"https://api.github.com/users/houndci/received_events","type":"User","site_admin":false},"body":"Line is too long. [94/80]","created_at":"2014-04-11T04:47:48Z","updated_at":"2014-04-11T04:47:48Z","html_url":"https://github.com/calblueprint/roots-of-success/pull/65#discussion_r11517981","pull_request_url":"https://api.github.com/repos/calblueprint/roots-of-success/pulls/65","_links":{"self":{"href":"https://api.github.com/repos/calblueprint/roots-of-success/pulls/comments/11517981"},"html":{"href":"https://github.com/calblueprint/roots-of-success/pull/65#discussion_r11517981"},"pull_request":{"href":"https://api.github.com/repos/calblueprint/roots-of-success/pulls/65"}}}}