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

fix: Return full name with apple sign in #194

Open
wants to merge 6 commits into
base: next
Choose a base branch
from

Conversation

TylerAHolden
Copy link

@TylerAHolden TylerAHolden commented Oct 12, 2021

Fixes #191 by adding the fullName object to the firebase User object that gets returned from the apple sign in method.

@TylerAHolden TylerAHolden changed the title fix: [WIP] return name with apple sign in fix: Return full name with apple sign in Oct 12, 2021
@moblizeitllc
Copy link

is this fixed?

@TylerAHolden
Copy link
Author

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.

@moblizeitllc
Copy link

just updating the capacitor plugin should get me the latest code correct?
if apple does it just once then what would be the best way to verify this?

@mauriceackel
Copy link

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.

@TylerAHolden
Copy link
Author

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.

@JoelAH
Copy link

JoelAH commented Feb 4, 2022

Can this be merged in & released? It impacts App submission

@parambidja
Copy link

Can this be merged? I'm having the same issue

@leonardoeckh
Copy link

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

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.

6 participants