{"action":"created","issue":{"url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/1","labels_url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/1/comments","events_url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/1/events","html_url":"https://github.com/anyTV/channel-growth-feature/issues/1","id":89490121,"number":1,"title":"Ask team lead or a devops to create repository so we can check the first code that will be pushed.","user":{"login":"pprmint","id":3439469,"avatar_url":"https://avatars.githubusercontent.com/u/3439469?v=3","gravatar_id":"","url":"https://api.github.com/users/pprmint","html_url":"https://github.com/pprmint","followers_url":"https://api.github.com/users/pprmint/followers","following_url":"https://api.github.com/users/pprmint/following{/other_user}","gists_url":"https://api.github.com/users/pprmint/gists{/gist_id}","starred_url":"https://api.github.com/users/pprmint/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pprmint/subscriptions","organizations_url":"https://api.github.com/users/pprmint/orgs","repos_url":"https://api.github.com/users/pprmint/repos","events_url":"https://api.github.com/users/pprmint/events{/privacy}","received_events_url":"https://api.github.com/users/pprmint/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-19T07:21:47Z","updated_at":"2015-06-19T07:26:52Z","closed_at":null,"body":"Unnecessary concatenation.\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/growth.js#L16\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/channel.js#L39\r\n\r\nDeclare the `data` in a global scope of the function. Also check if `data` is type of string so you can filter if there are any errors.\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/growth.js#L13\r\n\r\nUse `util.get_data()` for `req.query` as well and to the same check. I've already explained this before.\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/growth.js#L18\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/channel.js#L19\r\n\r\nNo need to declare `query` as empty string. You can just use the raw query directly in the mysql call instead of using a variable as storage.\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/channel.js#L23\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/channel.js#L39\r\n\r\nCheck if req.params.id exist\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/controllers/user.js#L12"},"comment":{"url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/comments/113406988","html_url":"https://github.com/anyTV/channel-growth-feature/issues/1#issuecomment-113406988","issue_url":"https://api.github.com/repos/anyTV/channel-growth-feature/issues/1","id":113406988,"user":{"login":"ravenjohn","id":3425521,"avatar_url":"https://avatars.githubusercontent.com/u/3425521?v=3","gravatar_id":"","url":"https://api.github.com/users/ravenjohn","html_url":"https://github.com/ravenjohn","followers_url":"https://api.github.com/users/ravenjohn/followers","following_url":"https://api.github.com/users/ravenjohn/following{/other_user}","gists_url":"https://api.github.com/users/ravenjohn/gists{/gist_id}","starred_url":"https://api.github.com/users/ravenjohn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ravenjohn/subscriptions","organizations_url":"https://api.github.com/users/ravenjohn/orgs","repos_url":"https://api.github.com/users/ravenjohn/repos","events_url":"https://api.github.com/users/ravenjohn/events{/privacy}","received_events_url":"https://api.github.com/users/ravenjohn/received_events","type":"User","site_admin":false},"created_at":"2015-06-19T07:26:52Z","updated_at":"2015-06-19T07:26:52Z","body":"get_user won't be called without req.params.id\r\nhttps://github.com/anyTV/channel-growth-feature/blob/master/config/router.js#L13\r\n\r\nno need to check for req.params.id"}}