{"action":"created","issue":{"url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/1","labels_url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/1/comments","events_url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/1/events","html_url":"https://github.com/crobertsbmw/deckofcards/pull/1","id":72212890,"number":1,"title":"Create a function to retrieve a specific request value from either POST ...","user":{"login":"larssorenson","id":8646771,"avatar_url":"https://avatars.githubusercontent.com/u/8646771?v=3","gravatar_id":"","url":"https://api.github.com/users/larssorenson","html_url":"https://github.com/larssorenson","followers_url":"https://api.github.com/users/larssorenson/followers","following_url":"https://api.github.com/users/larssorenson/following{/other_user}","gists_url":"https://api.github.com/users/larssorenson/gists{/gist_id}","starred_url":"https://api.github.com/users/larssorenson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/larssorenson/subscriptions","organizations_url":"https://api.github.com/users/larssorenson/orgs","repos_url":"https://api.github.com/users/larssorenson/repos","events_url":"https://api.github.com/users/larssorenson/events{/privacy}","received_events_url":"https://api.github.com/users/larssorenson/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-30T16:27:25Z","updated_at":"2015-04-30T20:25:53Z","closed_at":"2015-04-30T20:20:50Z","pull_request":{"url":"https://api.github.com/repos/crobertsbmw/deckofcards/pulls/1","html_url":"https://github.com/crobertsbmw/deckofcards/pull/1","diff_url":"https://github.com/crobertsbmw/deckofcards/pull/1.diff","patch_url":"https://github.com/crobertsbmw/deckofcards/pull/1.patch"},"body":"...or GET depending on the request's method\r\n\r\nHey!\r\n\r\nI love the API, it's a really cool idea and I think well implemented! When I was looking through the code, I noticed you had some redundant code when you were retrieving a variable from request.POST or request.GET, so I just pulled it into a separate function and set up shuffle and draw to use it.\r\n\r\nJust thought you might like this, if you expand the API it should make the code more consistent and easier to adjust in the future.\r\n\r\nThanks,\r\nLars"},"comment":{"url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/comments/97955773","html_url":"https://github.com/crobertsbmw/deckofcards/pull/1#issuecomment-97955773","issue_url":"https://api.github.com/repos/crobertsbmw/deckofcards/issues/1","id":97955773,"user":{"login":"crobertsbmw","id":5834142,"avatar_url":"https://avatars.githubusercontent.com/u/5834142?v=3","gravatar_id":"","url":"https://api.github.com/users/crobertsbmw","html_url":"https://github.com/crobertsbmw","followers_url":"https://api.github.com/users/crobertsbmw/followers","following_url":"https://api.github.com/users/crobertsbmw/following{/other_user}","gists_url":"https://api.github.com/users/crobertsbmw/gists{/gist_id}","starred_url":"https://api.github.com/users/crobertsbmw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/crobertsbmw/subscriptions","organizations_url":"https://api.github.com/users/crobertsbmw/orgs","repos_url":"https://api.github.com/users/crobertsbmw/repos","events_url":"https://api.github.com/users/crobertsbmw/events{/privacy}","received_events_url":"https://api.github.com/users/crobertsbmw/received_events","type":"User","site_admin":false},"created_at":"2015-04-30T20:25:53Z","updated_at":"2015-04-30T20:25:53Z","body":"Thanks. I haven't really had any formal training when it comes to python. This may be a noobie question, but is there a reason you defined the function beginning with an underscore?"}}