-
Notifications
You must be signed in to change notification settings - Fork 129
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
fix: Return full name with apple sign in #194
base: next
Are you sure you want to change the base?
Conversation
is this fixed? |
Yea this fixes it for me but just be wary of how it behaves... You'll get fullName back with the firebase.User object ONLY the first time you authenticate with apple. After that, Apple never sends you the name again. |
just updating the capacitor plugin should get me the latest code correct? |
Hi everyone: I'm currently also facing the issue that the display name is not provided when signing in with Apple. I checked the code of this PR and IMO this is rather a workaround than a fix and I don't know whether that should be included in the library. I don't want to offend anyone here and if this workaround fixes the issue in your use case that's great! I just think the issue should be fixed in the Firebase SDK instead of having a special workaround for only the Apple signin logic in this plugin. |
Yea this is kind of a work-around but given that it's a required feature of this library in order to submit to the iOS store it seems like reason enough to add a temporary patch. I haven't seen any movement on the Firebases SDK so it doesn't seem like waiting for them to add that feature is the best move. @mauriceackel nonetheless, I added the PR so others like yourself have something to work off of even if it doesn't get merged. |
Can this be merged in & released? It impacts App submission |
Can this be merged? I'm having the same issue |
Hey Tyler how i can use your branch? I already tried to install yarn add git+https://github.com/TylerAHolden/capacitor-firebase-auth.git but it doesn't recognize the module when importing |
Fixes #191 by adding the fullName object to the firebase User object that gets returned from the apple sign in method.