{"action":"created","issue":{"url":"https://api.github.com/repos/mromphf/blackjack/issues/1","labels_url":"https://api.github.com/repos/mromphf/blackjack/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/mromphf/blackjack/issues/1/comments","events_url":"https://api.github.com/repos/mromphf/blackjack/issues/1/events","html_url":"https://github.com/mromphf/blackjack/issues/1","id":94853016,"number":1,"title":"Application JavaScript should occupy only one global variable","user":{"login":"brettmclean","id":4391554,"avatar_url":"https://avatars.githubusercontent.com/u/4391554?v=3","gravatar_id":"","url":"https://api.github.com/users/brettmclean","html_url":"https://github.com/brettmclean","followers_url":"https://api.github.com/users/brettmclean/followers","following_url":"https://api.github.com/users/brettmclean/following{/other_user}","gists_url":"https://api.github.com/users/brettmclean/gists{/gist_id}","starred_url":"https://api.github.com/users/brettmclean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brettmclean/subscriptions","organizations_url":"https://api.github.com/users/brettmclean/orgs","repos_url":"https://api.github.com/users/brettmclean/repos","events_url":"https://api.github.com/users/brettmclean/events{/privacy}","received_events_url":"https://api.github.com/users/brettmclean/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-14T02:21:12Z","updated_at":"2015-07-14T03:18:02Z","closed_at":"2015-07-14T03:15:50Z","body":"The app should avoid placing more than one variable in the JavaScript global namespace. It currently exposes more functions this way than are necessary. \r\n\r\nThe fact that the user has access to the `dealerScore` and `playerScore` variables would normally be cause for alarm (they could cheat and edit these values using their browser's developer tools). In this case the values are always set by the server, but it's good to hide sensitive things like this from the user's view."},"comment":{"url":"https://api.github.com/repos/mromphf/blackjack/issues/comments/121121340","html_url":"https://github.com/mromphf/blackjack/issues/1#issuecomment-121121340","issue_url":"https://api.github.com/repos/mromphf/blackjack/issues/1","id":121121340,"user":{"login":"mromphf","id":8743051,"avatar_url":"https://avatars.githubusercontent.com/u/8743051?v=3","gravatar_id":"","url":"https://api.github.com/users/mromphf","html_url":"https://github.com/mromphf","followers_url":"https://api.github.com/users/mromphf/followers","following_url":"https://api.github.com/users/mromphf/following{/other_user}","gists_url":"https://api.github.com/users/mromphf/gists{/gist_id}","starred_url":"https://api.github.com/users/mromphf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mromphf/subscriptions","organizations_url":"https://api.github.com/users/mromphf/orgs","repos_url":"https://api.github.com/users/mromphf/repos","events_url":"https://api.github.com/users/mromphf/events{/privacy}","received_events_url":"https://api.github.com/users/mromphf/received_events","type":"User","site_admin":false},"created_at":"2015-07-14T03:18:02Z","updated_at":"2015-07-14T03:18:02Z","body":"Merged. Thanks for pointing out that oversight, and thanks for fixing it!"}}