Closed deepanchal closed 1 year ago
Note: This screen isn't connected in UI. Its a standalone screen for now. Once all links are working, I will connect it in AuthenticateScreen
AuthenticateScreen
As per @arroyoDev, we should make the following changes for MVP:
I need help cleaning up
https://crisiscleanup.org/survivor
Register Now
https://crisiscleanup.org/register
If you wish to test it out locally, please apply the following patch
diff --git a/feature/authentication/src/main/java/com/crisiscleanup/feature/authentication/ui/AuthenticateScreen.kt b/feature/authentication/src/main/java/com/crisiscleanup/feature/authentication/ui/AuthenticateScreen.kt index e0eae11b..bcdd7954 100644 --- a/feature/authentication/src/main/java/com/crisiscleanup/feature/authentication/ui/AuthenticateScreen.kt +++ b/feature/authentication/src/main/java/com/crisiscleanup/feature/authentication/ui/AuthenticateScreen.kt @@ -145,11 +145,17 @@ private fun AuthenticateScreen( onCloseScreen, ) } else { - LoginScreen( - authState, - openForgotPassword = openForgotPassword, - openEmailMagicLink = openEmailMagicLink, - closeAuthentication = onCloseScreen, +// LoginScreen( +// authState, +// openForgotPassword = openForgotPassword, +// openEmailMagicLink = openEmailMagicLink, +// closeAuthentication = onCloseScreen, +// ) + RootLoginScreen( +// authState, +// openForgotPassword, +// openEmailMagicLink, +// closeAuthentication, ) Spacer(modifier = Modifier.weight(1f))
pr72.webm
Ref: #7
Please check the type of change your PR introduces:
Description
Note: This screen isn't connected in UI. Its a standalone screen for now. Once all links are working, I will connect it in
AuthenticateScreen
As per @arroyoDev, we should make the following changes for MVP:
I need help cleaning up
should navigate tohttps://crisiscleanup.org/survivor
Register Now
should open browser and navigate tohttps://crisiscleanup.org/register
If you wish to test it out locally, please apply the following patch
Changelog
Screenshots
pr72.webm
Related Issue
Ref: #7
Pull Request Type
Please check the type of change your PR introduces:
Checklist
Does this introduce a breaking change?