Move credential request logic out of LoginButton
@conniecliu ya está trabajando en esto.
Desde el 29/9/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
LoginButton currently does more than display a button. When clicked, it builds a Google sign-in request, opens Android's credential picker, receives the credential, handles exceptions, and forwards results to the LoginViewModel, which is doing too much for a UI composable.
The current ViewModel handles the Firebase sign-in, email validation, backend login, and navigation. Responsibilities between the ViewModel and View should be reorganized:
LoginButton: accept anonClickcallback and an enabled/loading state- Factor out the business logic (credential picker, exceptions, building a request)
LoginViewModel: process credentials, validation, backend, Firebase
- Lenguaje dominante
- Kotlin
- Estrellas
- 2
- Forks
- 0
- Merge medio
- 18 d 4 h
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cuappdev/uplift-android
-
Profile picture not being updated/synced after onboardingPosiblemente ocupada @conniecliu la tomó hace 22 días. Abierto
cuappdev/uplift-android#119 · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cuappdev/uplift-android#114 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cuappdev/uplift-android#112 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cuappdev/uplift-android#111 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
cuappdev/uplift-android#110 ·
Todos los issues de cuappdev/uplift-android
Issues similares
-
LiquidGlass / LiquidGlassContainer (iOS): padded content measured at the outer width and overflows by the paddingPosiblemente ocupada @aleclarson la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
NativeScript/NativeScript#11561 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
navikt/mock-oauth2-server#1129 ·
Los mantenedores suelen responder en 1 día
-
Expose `close` on SsrEngineAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
-
unconfirmed
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
thunderbird/thunderbird-android#11778 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
software-mansion/enriched-markdown#1042 ·
Los mantenedores suelen responder en 2 días