Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a default to response in FacebookOAuth.do_auth #12

Merged
merged 2 commits into from Jun 13, 2013

Conversation

san-mate
Copy link
Collaborator

The response default can be a dict because, inside the method, you are using this similar as a dict, but in future cases that you need the Response object this will fail.

omab added a commit that referenced this pull request Jun 13, 2013
Added a default to response in FacebookOAuth.do_auth
@omab omab merged commit fa49566 into omab:master Jun 13, 2013
@omab
Copy link
Owner

omab commented Jun 13, 2013

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants