-
Notifications
You must be signed in to change notification settings - Fork 101
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
Documentation is lacking #33
Comments
What are your issues? |
@jmaha I am also struggling - Did you manage to get this to work ? |
@NdaJunior I got it as far as getting Sign in with Apple working from a web browser (after digging through other issue reports to determine that CLIENT_ID is the Service ID you create in Apple's developer portal). However, I was unable to also support signing in from an iOS app. The credentials from the mobile app have to be validated using the App ID--the Service ID tied to the App ID doesn't work. It seems like omniauth-apple doesn't support this (hard to tell as it's pretty much undocumented, and it wasn't clear from the source code). We finally ended up dropping omniauth-apple and sunsetting Facebook login, eliminating the requirements to support Sign in with Apple. Not a great solution, but it's what made sense for us. |
@jmaha I've managed to get as far as being redirected from my rails app to the apple ID login where you grant permissions for your app. I'm facing an issue now with the redirect back to my rails app where it complains about:
Did you ever experience this ? |
Apple has notoriously fluffy docs, and this repo could solve SO MUCH PAIN by just listing the steps involved |
Have you looked https://github.com/nhosoya/omniauth-apple-sample ? |
I just wrote an apple walkthrough for the readme #62 |
This project needs thorough documentation. I've been trying to get it to work for the last week with no success. Comparing the source code to omniauth-facebook, omniauth-apple appears to work quite differently.
The text was updated successfully, but these errors were encountered: