-
Notifications
You must be signed in to change notification settings - Fork 6
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
Agregar verificacion al form de Login #45
Comments
Se soluciono el problema utilizando expresiones regulares con JavaScript puro, de la siguiente forma
Al mismo tiempo se realizaron cambios en componentes tales como ButtonPrimary, agregandole dos props, un hancleClick y un typeOfButton el cual indicaria el tipo de boton que se quiere colocar. Por el momento de prescindio de usar Formik o Yup debido a que el proyecto de momento no es tan complejo, sin embargo, de querer implementarlo me dicen y lo implemento. |
@Durphan Lo ideal sería que ese código esté solamente en el PR, mientras que en la descripción del mismo, agregar la razón del cambio, así como por qué se fue por una rgx en lugar de otra estrategia. Asociado a esto, un mini resumen de las pruebas hechas para comprobar dicho cambio :) De esta forma, el issue queda con el detalle concreto de lo que se debe solucionar, mientras que en el PR la solución del mismo |
Se deberia de agregar un sistema de verificacion al momento de intentar logearse, el mismo debe de ser uno simple para usuario y contraseña que tenga lo siguiente:
Gmail: Verificar que el string que coloque el usuario sea un gmail
Contraseña: Verificar que la contraseña tenga un minimo de 4 caracteres y un numero
Si no se cumple una condicion entonces debe aparecer un mensaje al usuario indicando que requisito no cumplio
The text was updated successfully, but these errors were encountered: