Skip to content

Conversation

@protocol86
Copy link
Contributor

Adds support for Facebook when autolinking an existing user to a new federated user. Tested with Google and Facebook auth credentials for now.

Adds support for Facebook when autolinking an exisiting user to a new federated user (Google).
NSString *existingFederatedProviderID = [self federatedAuthProviderFromProviders:providers];
// Set of providers which can be auto-linked
NSSet *supportedProviders =
[NSSet setWithObjects:FIRGoogleAuthProviderID,FIRFacebookAuthProviderID, nil];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: space after first comma

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, thanks

[googleProviderUI signInWithDefaultValue:emailHint
[authProviderUI signOut];
[authProviderUI signInWithDefaultValue:emailHint
presentingViewController:presentingViewController
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: fix indents

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, thanks

@protocol86 protocol86 merged commit 9bf67eb into anonymous_updgrade_master Feb 16, 2018
protocol86 added a commit that referenced this pull request Jul 17, 2018
* Facebook support for email_in_user_when_upgrading

Adds support for Facebook when autolinking an exisiting user to a new federated user (Google).

* Addresses comments
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