Skip to content

wip: minimal firebase auth ui migration #1

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 21 commits into
base: version-9.0.0
Choose a base branch
from
Draft

wip: minimal firebase auth ui migration #1

wants to merge 21 commits into from

Conversation

demolaf
Copy link
Owner

@demolaf demolaf commented Sep 3, 2025

Hey there! So you want to contribute to FirebaseUI? Before you file this pull request, follow these steps:

  • Read the contribution guidelines.
  • Run ./gradlew check to ensure the Travis build passes.
  • If this has been discussed in an issue, make sure to mention the issue number here. If not, go file an issue about this to make sure this is a desirable change.
  • If this is a new feature please co-ordinate with someone on FirebaseUI-iOS to make sure that we can implement this on both platforms and maintain feature parity.

@demolaf demolaf marked this pull request as draft September 3, 2025 07:46
demolaf added 20 commits September 4, 2025 01:29
@demolaf
refactor: improve AuthUI singleton pattern with proper initialization
4c5ef1e 
- Separate instance creation from config in AuthUI singleton
- Add initialize() method that creates AuthUI with config and caches it
- Update getInstance() to retrieve cached instances or throw if not initialized
- Create AuthUIScreen composable to replace direct initialization
- Move provider configuration from MainView to MainActivity
- Rename EmailSignInButton to EmailAndPasswordSignInButton
- Disable auto-select in Google Sign-In for better UX
@demolaf
chore: code cleanup
eb83383
@demolaf
wip: email and password sign in, use singleton pattern, navigation 3 …
4c9ab03 
…apis
@demolaf
fix: use canPop to handle exiting AuthUIScreen
8b5c308
@demolaf
refator: use Views instead of Screens, remove IdpConfig and replaced …
03340f6 
…with AuthProvider sealed classes
@demolaf
wip: nested navdisplay causing parent back handler to trigger on swip…
61bd846 
…e back gesture
@demolaf
refactor: reorganize AuthUI package structure and update imports
49b51e5
@demolaf
refactor: complete auth compose module restructuring and implementation
6a8038c
@demolaf
feat: complete email auth integration with proper state management
285eb8c
@demolaf
feat: add comprehensive Google sign-in testing with Firebase emulator…
c04877f 
… integration
@demolaf
remove TestCustomCredential and use mockito directly
367f396
@demolaf
add CI to test e2e google sign in flow
51a373d
@demolaf
fix: ci issue no firebase.json or .firebaserc files found
fb230eb
@demolaf
fix: ci issue run only unit tests
f30e28d
@demolaf
remove failing CI tests
bfcd532
@demolaf
without starting firebase emulator CI fails
9ecc376
@demolaf
without starting firebase emulator CI fails
07c5dc3
@demolaf
without starting firebase emulator CI fails
365e32f
@demolaf
with starting firebase emulator CI succeeds
dedee85
@demolaf
chore: clearer ci steps name
d146591
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@demolaf