Skip to content

1) Feature/mobile login #23

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

Merged
merged 3 commits into from
Jan 3, 2025
Merged

1) Feature/mobile login #23

merged 3 commits into from
Jan 3, 2025

Conversation

joelcamera
Copy link
Collaborator

@joelcamera joelcamera commented Dec 9, 2024

Login mobile

  • Se agrega la vista de login para mobile, que sirve de webview para la aplicacion delta-emisor.
  • Se le agrega un header simple ya que si se sale del flujo de login en la mobile app no se puede volver a él (es un login que le da datos del usuario a la mobile app y no hace nada mas, tiene una url especial que no se puede acceder desde la webapp).

Joel added 3 commits October 28, 2024 15:54
…a poder loguearse en la aplicacion delta emisor.
…irecciones a otras paginas de la webapp.

Esto se da por el flujo especial que se necesita para ese login (se loguea, consigue el usuario, las lanchas del mismo y lo devuelve a la aplicacion flutter). Si se sale de ese flujo, no se puede volver desde la webapp a este login y queda en un estado inconsistente.
@joelcamera joelcamera requested a review from lhazuca December 9, 2024 15:22
@joelcamera joelcamera changed the title Feature/mobile login 1) Feature/mobile login Dec 11, 2024
.required();


const LoginOKComponent : React.FC = () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

hace falta usar React.FC?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No se de cuanto tiempo disponemos para pulir pero unas abstracciones sobre algunos componentes ayudarian mucho a la legibilidad de esto

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Es verdad que esta re duro, te cuento lo que hice: Agarré el login que habia, le saque algunas cosas del flujo comun de login web, y le agregue el useEffect de la linea 50, que es el que chequea que si esta logueado pasa los datos del usuario por el handler de flutter. El resto esta igual porque me falta contexto del login y no quiero que algo se rompa. Claramente es horrible ese componente así como esta....
(y por todo esto está el React.FC, porque viene del otro login :P )

@lhazuca lhazuca merged commit 81be0b8 into main Jan 3, 2025
1 check passed
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.

2 participants