-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: OAuth Providers (Twitter, GitHub, Microsoft, Yahoo, Apple), Generic OAuth Provider #2255
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
Conversation
- unit tests covering google sign in, anonymous upgrade and error states - updated google auth dependencies
- flows with scopes/customParameters - apple locale override support for button - providerId, scopes, customParameters, with customizable button (label/icon/color) - update firebase brand auth logo and app icon - translations for sign in with LINE - generic/custom user define oauth example with sign in with discord.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Logs to be removed but looking good otherwise
| MICROSOFT("microsoft.com", providerName = "Microsoft"), | ||
| YAHOO("yahoo.com", providerName = "Yahoo"), | ||
| APPLE("apple.com", providerName = "Apple"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are those not considered social provider here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes actually
| // Log.d("GoogleSignIn", "Successfully authorized scopes: ${provider.scopes}") | ||
| } catch (e: Exception) { | ||
| // Log.w("GoogleSignIn", "Failed to authorize scopes: ${provider.scopes}", e) | ||
| // Continue with sign-in even if scope authorization fails |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remove the 3 commented logs here
| // Log.d( | ||
| // "OAuthSignIn", | ||
| // "Successfully obtained OAuth credential for ${provider.providerName}" | ||
| // ) | ||
| // The user is already signed in via startActivityForSignInWithProvider/startActivityForLinkWithProvider |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
REmove logs
P6 - Implement OAuth flows with scopes/customParameters; Apple supports locale.
P7 - Generic handler by providerId, scopes, customParameters, with customizable button (label/icon/color).
sign_in_with_github_and_LINE.webm
sign_in_with_generic_oauth_discord.webm