{"action":"created","issue":{"url":"https://api.github.com/repos/MystK/delta-bot-three/issues/41","repository_url":"https://api.github.com/repos/MystK/delta-bot-three","labels_url":"https://api.github.com/repos/MystK/delta-bot-three/issues/41/labels{/name}","comments_url":"https://api.github.com/repos/MystK/delta-bot-three/issues/41/comments","events_url":"https://api.github.com/repos/MystK/delta-bot-three/issues/41/events","html_url":"https://github.com/MystK/delta-bot-three/issues/41","id":180466420,"number":41,"title":"Testing mode","user":{"login":"roastchicken","id":524709,"avatar_url":"https://avatars.githubusercontent.com/u/524709?v=3","gravatar_id":"","url":"https://api.github.com/users/roastchicken","html_url":"https://github.com/roastchicken","followers_url":"https://api.github.com/users/roastchicken/followers","following_url":"https://api.github.com/users/roastchicken/following{/other_user}","gists_url":"https://api.github.com/users/roastchicken/gists{/gist_id}","starred_url":"https://api.github.com/users/roastchicken/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/roastchicken/subscriptions","organizations_url":"https://api.github.com/users/roastchicken/orgs","repos_url":"https://api.github.com/users/roastchicken/repos","events_url":"https://api.github.com/users/roastchicken/events{/privacy}","received_events_url":"https://api.github.com/users/roastchicken/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2016-10-01T16:43:38Z","updated_at":"2016-10-03T01:03:51Z","closed_at":null,"body":"Currently there are hard-coded strings in the code checking to see if the author is `mystk` to circumvent tests for giving yourself a delta. I think there is better way of doing this, instead of requiring people to replace these occurrences with their own username.\r\n\r\nTwo possible ways of handling this that I've though of so far:\r\n\r\n- Add a new key to credentials.json: `\"testingUsername\": \"myUsernameToUseForTesting\"`\r\n- Add some way to use 'testing mode' where checks for authorship are not made\r\n\r\nThe downside to the `testingUsername` approach is that this may not be wanted in production. I guess it could be set to a blank string or something like that to disable it.\r\n\r\nThe downside to the testing mode is that the author check may be a check that fails when the code is changed, and if testing mode is enabled then this bug may not be noticed until it is put into production.\r\n\r\nFeel free to critique these ideas and/or suggest other ways of handling this."},"comment":{"url":"https://api.github.com/repos/MystK/delta-bot-three/issues/comments/251008751","html_url":"https://github.com/MystK/delta-bot-three/issues/41#issuecomment-251008751","issue_url":"https://api.github.com/repos/MystK/delta-bot-three/issues/41","id":251008751,"user":{"login":"MystK","id":7399140,"avatar_url":"https://avatars.githubusercontent.com/u/7399140?v=3","gravatar_id":"","url":"https://api.github.com/users/MystK","html_url":"https://github.com/MystK","followers_url":"https://api.github.com/users/MystK/followers","following_url":"https://api.github.com/users/MystK/following{/other_user}","gists_url":"https://api.github.com/users/MystK/gists{/gist_id}","starred_url":"https://api.github.com/users/MystK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MystK/subscriptions","organizations_url":"https://api.github.com/users/MystK/orgs","repos_url":"https://api.github.com/users/MystK/repos","events_url":"https://api.github.com/users/MystK/events{/privacy}","received_events_url":"https://api.github.com/users/MystK/received_events","type":"User","site_admin":false},"created_at":"2016-10-03T01:03:51Z","updated_at":"2016-10-03T01:03:51Z","body":"Up to you, to be honest. Doesn't matter to me. As long as the approach you make doesn't affect the --debug flag. ;)"}}
{"actor":{"display_login":"MystK"}}