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

remove extra step for parsing facebook response as was returning nil #175

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mlennie
Copy link

@mlennie mlennie commented Sep 5, 2017

This fixes this issue #170
Extra Facebook parse step was causing issue

@lethunder
Copy link

Hey @mlennie

did you succeed to use facebook connector?

i have this on my side

Missing redirect_uri parameter

Thanks in advance

@mlennie
Copy link
Author

mlennie commented Sep 28, 2017

@lethunder yes my PR fixes the problem for me. d484946

@lethunder
Copy link

@mlennie thanks

@lethunder
Copy link

Hey @mlennie ,
i still have issue with facebook even with your PR
i have this
Missing redirect_uri parameter.

this is my fork url
https://github.com/lethunder/omnicontacts

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.

2 participants