)]}'
{"app/js/services/view.js":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a2a9ee0b565e4c936a73b1dd9b64314593c4a2d3","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  var periodEnd \u003d moment();"},{"line_number":41,"context_line":"  periodEnd.startOf(\u0027hour\u0027);"},{"line_number":42,"context_line":"  periodEnd.add(\u0027hour\u0027);"},{"line_number":43,"context_line":"  periodEnd \u003d periodEnd.toDate();"},{"line_number":44,"context_line":"  var periodOptions \u003d ["},{"line_number":45,"context_line":"    moment.duration({ hours: 1 }),"}],"source_content_type":"text/javascript","patch_set":5,"id":"9abb7d3a_7bcb0f54","line":42,"range":{"start_line":42,"start_character":16,"end_line":42,"end_character":22},"updated":"2016-05-30 03:00:26.000000000","message":"I guess you wanted to be here is \n\n periodEnd.add(1, \u0027hours\u0027);\n\n?","commit_id":"eb2032c5d56c0696e42e332948d07ef2244079e1"},{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"cf3ac0a062787794764248a9e9f729c0de60d5a9","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  var periodEnd \u003d moment();"},{"line_number":41,"context_line":"  periodEnd.startOf(\u0027hour\u0027);"},{"line_number":42,"context_line":"  periodEnd.add(\u0027hour\u0027);"},{"line_number":43,"context_line":"  periodEnd \u003d periodEnd.toDate();"},{"line_number":44,"context_line":"  var periodOptions \u003d ["},{"line_number":45,"context_line":"    moment.duration({ hours: 1 }),"}],"source_content_type":"text/javascript","patch_set":5,"id":"7aa08908_756ff391","line":42,"in_reply_to":"9abb7d3a_68d439eb","updated":"2016-06-14 19:34:24.000000000","message":"Done","commit_id":"eb2032c5d56c0696e42e332948d07ef2244079e1"},{"author":{"_account_id":4656,"name":"Sean M. Collins","email":"sean@coreitpro.com","username":"scollins"},"change_message_id":"cf3ac0a062787794764248a9e9f729c0de60d5a9","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  var periodEnd \u003d moment();"},{"line_number":41,"context_line":"  periodEnd.startOf(\u0027hour\u0027);"},{"line_number":42,"context_line":"  periodEnd.add(\u0027hour\u0027);"},{"line_number":43,"context_line":"  periodEnd \u003d periodEnd.toDate();"},{"line_number":44,"context_line":"  var periodOptions \u003d ["},{"line_number":45,"context_line":"    moment.duration({ hours: 1 }),"}],"source_content_type":"text/javascript","patch_set":5,"id":"7aa08908_35cbaba5","line":42,"in_reply_to":"9abb7d3a_7bcb0f54","updated":"2016-06-14 19:34:24.000000000","message":"Done","commit_id":"eb2032c5d56c0696e42e332948d07ef2244079e1"},{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"2c3c82115d1bda11d3133ee3ac04a13f61c54ef8","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  var periodEnd \u003d moment();"},{"line_number":41,"context_line":"  periodEnd.startOf(\u0027hour\u0027);"},{"line_number":42,"context_line":"  periodEnd.add(\u0027hour\u0027);"},{"line_number":43,"context_line":"  periodEnd \u003d periodEnd.toDate();"},{"line_number":44,"context_line":"  var periodOptions \u003d ["},{"line_number":45,"context_line":"    moment.duration({ hours: 1 }),"}],"source_content_type":"text/javascript","patch_set":5,"id":"9abb7d3a_68d439eb","line":42,"range":{"start_line":42,"start_character":16,"end_line":42,"end_character":22},"in_reply_to":"9abb7d3a_7bcb0f54","updated":"2016-06-01 23:17:59.000000000","message":"Nit: we can also chain methods here, e.g.:\n\n    var periodEnd \u003d moment()\n        .startOf(\u0027hour\u0027)\n        .add(1, \u0027hours\u0027)\n        .toDate();","commit_id":"eb2032c5d56c0696e42e332948d07ef2244079e1"},{"author":{"_account_id":17001,"name":"Tim Buckley","email":"timothy.jas.buckley@hpe.com","username":"timothy.jas.buckley"},"change_message_id":"2c3c82115d1bda11d3133ee3ac04a13f61c54ef8","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    },"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    periodStart: function() {"},{"line_number":120,"context_line":"      var start \u003d moment(periodEnd);"},{"line_number":121,"context_line":"      start.startOf(\u0027hour\u0027);"},{"line_number":122,"context_line":"      return start.subtract(selectDuration()).toDate();"},{"line_number":123,"context_line":"    },"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    periodOptions: function() {"}],"source_content_type":"text/javascript","patch_set":5,"id":"9abb7d3a_88a4fd4a","line":122,"range":{"start_line":120,"start_character":6,"end_line":122,"end_character":55},"updated":"2016-06-01 23:17:59.000000000","message":"We should probably do this mutation while updating the periodEnd value (line #112) so we consistently honor the duration field (periodStart should be a computed read-only value).\n\nSame nit as above as well","commit_id":"eb2032c5d56c0696e42e332948d07ef2244079e1"}]}