feat: Login with Email connection with RootLoginPage

[deepanchal]

Description

• fix(authentication): remove duplicate var assign for isInvalidEmail in AuthenticationViewModel (a3c7daa1)
• refactor(authentication): split links into vars on RootLoginScreen (cb3fe739)
• feat(auth): pass authState in RootLoginScreen, use emptyAccountData for screen preview (f586e766)
• fix(auth): wrap RootLoginScreen elements in Column as it was generating incorrect preview (163b227f)
• feat(auth): disable phone login & volunteer with org buttons on RootLoginScreen (2dd97ebc)
• feat(appnav): add loginWithEmailRoute constant (19d1c735)
• fix(appnav): make login_with_email nested under auth_route (3f77274b)
• feat(auth): add LoginWithDifferentMethod composable in AuthenticateScreen (f174b4f1)
• feat(auth): add LoginWithEmailNavigation with nav controller & graph builder for loginWithEmail under navigation (81e345f3)
• feat(auth/ui): add RootAuthRoute composable, add openLoginWithEmail param in RootLoginScreen (87eafa8a)
• fix(auth/ui): pass through onBack handler to LoginScreen (6d06f45e)
• fix(auth/ui): use LoginWithDifferentMethod composable on LoginScreen (13d6ed46)
• feat(app): fix routes in CrisisCleanupAuthNavHost (e4d1a801)
• fix(auth/navigation): link authGraph to RootAuthRoute from AuthRoute (c8a2c0b8)
• feat(auth): add loginUsingDifferentMethod text to strings.xml, update usage in LoginUsingDifferentMethod (06bfbc8f)
• feat(auth): add preview for LoginUsingDifferentMethod (11f71c5c)
• refactor(auth): use correct import of RouteConstant in LoginWithEmailNavigation.kt (bfc91e5e)
• refactor(auth): rename RootLoginScreen to RootAuthScreen, update usages (9b8ce57a)
• feat(auth): add RootAuthViewModel with authState (e2961c69)
• fix(auth): fix profile btn not showing AuthenticatedScreen, use AuthenticatedScreen using authState from viewmodel in RootAuthScreen (2725a60b)
• fix(auth): pass through missing closeAuthentication handler to RootAuthScreen to close AuthenticatedScreen on back btn press (8c6f5674)
• fix(auth): remove AuthenticatedScreen usage from AuthenticateScreen as it's now used in RootAuthScreen (5e893b6a)
• refactor(auth): rename LoginScreen to LoginWithEmail (07880ee4)
• refactor(auth): move reusable auth composables to AuthComposables.kt (66bd079b)
• refactor(auth): remove unused AuthRoute import in AuthNavigation.kt (45f50c48)
• refactor(auth): rename AuthRoute to LoginWithEmailRoute (eb11d744)
• refactor(auth): merge AuthenticateScreen into LoginWithEmailRoute to keep it simple (d0f56b5a)
• refactor(auth): rename AuthenticateScreen.kt to LoginWithEmailScreen.kt (677b0028)
• refactor(auth): move AuthenticatedScreen from LoginWithEmailScreen.kt to RootAuthScreen.kt (1ea131e9)
• fix(auth): pass forgotPasswordScreen & emailLoginLinkScreen as nestedGraph in loginWithEmailScreen to fix broken forgot password button (a5c136df)

Screenshots

pr75.webm

N/A

Related Issue

Pull Request Type

Please check the type of change your PR introduces:

• [x] Bugfix
• [x] Feature
• [ ] Code style update (formatting, renaming)
• [x] Refactoring (no functional changes, no api changes)
• [ ] Build related changes
• [ ] Documentation changes

Checklist

• [x] My code follows the code style of this project.
• [ ] I have updated the documentation.
• [ ] I have added tests to cover my changes.
• [x] All new and existing tests passed.

Does this introduce a breaking change?

• [ ] Yes
• [x] No

[hueachilles]

What seems to be the issue? Seems like it works like you were expecting no?

[deepanchal]

What seems to be the issue? Seems like it works like you were expecting no?

See what happens when I try to logout When you click profile icon, it takes you back to root page https://github.com/CrisisCleanup/crisiscleanup-android/blob/1b47c361353ed31ce2bd2e802931b708a7cc58d7/app/src/main/java/com/crisiscleanup/ui/CrisisCleanupApp.kt#L151-L162

[deepanchal]

@hueachilles Here are my new changes: https://github.com/CrisisCleanup/crisiscleanup-android/pull/75/files/51eb431c8c968dfe576308bdf9acd6297862e4c9..2c55a525ebd7ebdaff9783cd617481b84e14e5e6