)]}'
{"app/js/controllers/tests.js":[{"author":{"_account_id":18001,"name":"Caio Carrara","email":"ccarrara@thoughtworks.com","username":"caiocarrara"},"change_message_id":"27af4a0f77535c3b2c7571ee559ea7a896b4d07e","unresolved":false,"context_lines":[{"line_number":5,"context_line":"/**"},{"line_number":6,"context_line":" * @ngInject"},{"line_number":7,"context_line":" */"},{"line_number":8,"context_line":"function TestsCtrl($http) {"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"  // ViewModel"},{"line_number":11,"context_line":"  var vm \u003d this;"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_365b4656","line":8,"updated":"2015-10-07 17:14:48.000000000","message":"Since the Controller is related to a specific resource, it seems be better to use singular resource name followed by \"Ctrl\". At least this pattern was used in others controllers.","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa5fab11d429c9875a90248b10e219e7154b665f","unresolved":false,"context_lines":[{"line_number":5,"context_line":"/**"},{"line_number":6,"context_line":" * @ngInject"},{"line_number":7,"context_line":" */"},{"line_number":8,"context_line":"function TestsCtrl($http) {"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"  // ViewModel"},{"line_number":11,"context_line":"  var vm \u003d this;"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_39cb3974","line":8,"in_reply_to":"7a2fa921_365b4656","updated":"2015-10-07 17:20:54.000000000","message":"Well this is where me sucking at naming things comes up, because I also want to do a per test view. (see the follow-on patch) this is the summary page which is why it\u0027s plural because there are multiple tests","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"},{"author":{"_account_id":18001,"name":"Caio Carrara","email":"ccarrara@thoughtworks.com","username":"caiocarrara"},"change_message_id":"27af4a0f77535c3b2c7571ee559ea7a896b4d07e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"  }"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"  vm.loadData \u003d function() {"},{"line_number":36,"context_line":"    $http.jsonp(\"http://localhost:5000/tests\", {"},{"line_number":37,"context_line":"      params: {"},{"line_number":38,"context_line":"        callback: \u0027JSON_CALLBACK\u0027"},{"line_number":39,"context_line":"      }"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_d91ec562","line":36,"updated":"2015-10-07 17:14:48.000000000","message":"Specially now we have HealthServices merged, seems better to create a new service function there and make the request to server-side api as the others is done.","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa5fab11d429c9875a90248b10e219e7154b665f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"  }"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"  vm.loadData \u003d function() {"},{"line_number":36,"context_line":"    $http.jsonp(\"http://localhost:5000/tests\", {"},{"line_number":37,"context_line":"      params: {"},{"line_number":38,"context_line":"        callback: \u0027JSON_CALLBACK\u0027"},{"line_number":39,"context_line":"      }"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_79af7194","line":36,"in_reply_to":"7a2fa921_d91ec562","updated":"2015-10-07 17:20:54.000000000","message":"Sure, I can respin this on that. This patch predated the addition of the HealthServices functions.","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"},{"author":{"_account_id":17995,"name":"Glauco Oliveira","email":"gvinici@thoughtworks.com","username":"glauco"},"change_message_id":"b9bb87f413055fe29cea08b3ee18d94dec0cb50a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"/**"},{"line_number":6,"context_line":" * @ngInject"},{"line_number":7,"context_line":" */"},{"line_number":8,"context_line":"function TestsCtrl($http) {"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"  // ViewModel"},{"line_number":11,"context_line":"  var vm \u003d this;"}],"source_content_type":"text/javascript","patch_set":7,"id":"7a2fa921_ea1b7671","line":8,"updated":"2015-10-08 13:37:00.000000000","message":"I believe it should be \u0027function TestsCtrl($http, healthService) {\u0027. Otherwise, the page will not be able to get any information from api.","commit_id":"ff144473d1127f6895b2e3356604c3d89641cd2a"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"5d514b4c8615b039c42c12bbfd3d0821b17b6d5c","unresolved":false,"context_lines":[{"line_number":5,"context_line":"/**"},{"line_number":6,"context_line":" * @ngInject"},{"line_number":7,"context_line":" */"},{"line_number":8,"context_line":"function TestsCtrl($http) {"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"  // ViewModel"},{"line_number":11,"context_line":"  var vm \u003d this;"}],"source_content_type":"text/javascript","patch_set":7,"id":"7a2fa921_f0904445","line":8,"in_reply_to":"7a2fa921_ea1b7671","updated":"2015-10-08 15:00:52.000000000","message":"Oops, good catch, I\u0027ll respin this shortly.","commit_id":"ff144473d1127f6895b2e3356604c3d89641cd2a"},{"author":{"_account_id":17995,"name":"Glauco Oliveira","email":"gvinici@thoughtworks.com","username":"glauco"},"change_message_id":"b9bb87f413055fe29cea08b3ee18d94dec0cb50a","unresolved":false,"context_lines":[{"line_number":33,"context_line":"  }"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"  vm.loadData \u003d function() {"},{"line_number":36,"context_line":"    healthService.getTests().then(function(response) {"},{"line_number":37,"context_line":"      vm.processData(response.data);"},{"line_number":38,"context_line":"    });"},{"line_number":39,"context_line":"  };"}],"source_content_type":"text/javascript","patch_set":7,"id":"7a2fa921_caa23afd","line":36,"updated":"2015-10-08 13:37:00.000000000","message":"Since this is not being injected, there will be a JavaScript error.","commit_id":"ff144473d1127f6895b2e3356604c3d89641cd2a"}],"app/js/on_config.js":[{"author":{"_account_id":17995,"name":"Glauco Oliveira","email":"gvinici@thoughtworks.com","username":"glauco"},"change_message_id":"638f2606718615d9d9bc4496fa797e0ace1276d6","unresolved":false,"context_lines":[{"line_number":20,"context_line":"      resolve: {"},{"line_number":21,"context_line":"        \"projectName\": function($stateParams) {"},{"line_number":22,"context_line":"            return $stateParams.projectName;"},{"line_number":23,"context_line":"        }"},{"line_number":24,"context_line":"    })"},{"line_number":25,"context_line":"    .state(\u0027tests\u0027, {"},{"line_number":26,"context_line":"        url: \u0027/tests\u0027,"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_b62f56f5","line":23,"updated":"2015-10-07 17:04:28.000000000","message":"missing \u0027}\u0027","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"fa5fab11d429c9875a90248b10e219e7154b665f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"      resolve: {"},{"line_number":21,"context_line":"        \"projectName\": function($stateParams) {"},{"line_number":22,"context_line":"            return $stateParams.projectName;"},{"line_number":23,"context_line":"        }"},{"line_number":24,"context_line":"    })"},{"line_number":25,"context_line":"    .state(\u0027tests\u0027, {"},{"line_number":26,"context_line":"        url: \u0027/tests\u0027,"}],"source_content_type":"text/javascript","patch_set":6,"id":"7a2fa921_79869114","line":23,"in_reply_to":"7a2fa921_b62f56f5","updated":"2015-10-07 17:20:54.000000000","message":"Oops, good catch. A sloppy rebase","commit_id":"b36df2cf2ca9a44fdd2d2720aca7b41ee967128f"}]}