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

Add sign in and sign up with Google to Android samples #67

Merged
merged 8 commits into from
May 7, 2024

Conversation

marinacoelho
Copy link
Collaborator

This PR adds a new folder to the android video samples folder. It contains the sample code for sign in and sign up with Google using Credential Manager.

Most of the new files are the same from the previous code sample (built for the anonymous authentication video), so focus on the review of:

  • build.gradle.kts
  • AccountService.kt
  • AccountServiceImpl.kt
  • AccountCenterButtons.kt
  • SignInScreen.kt
  • SignInViewModel.kt
  • SignUpScreen.kt
  • SignUpViewModel.kt

This is how the new UI looks like:

Sign_in_with_Google
Sign_up_with_Google
Credential_Manager

@marinacoelho marinacoelho changed the title Add sign in and sign up with Google Add sign in and sign up with Google to Android samples May 3, 2024
@peterfriese peterfriese self-requested a review May 3, 2024 13:27
Copy link
Collaborator

@peterfriese peterfriese left a comment

Choose a reason for hiding this comment

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

I'd prefer if we used the same directory structure as for Apple:

firebase-video-samples/fundamentals/apple at main · FirebaseExtended/firebase-video-samples

I assume this PR uses the new (?) Sign in with Google, so I'd be OK with "auth-sign-in-with-google".

@marinacoelho marinacoelho merged commit 8e0e8bc into main May 7, 2024
3 checks passed
@marinacoelho marinacoelho deleted the marina/add-SiWG branch May 7, 2024 15:04
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.

None yet

3 participants