Tareas #19
Tareas #19
Conversation
There was a problem hiding this comment.
están perfectas las tareas (incluida la 4ta con la que tenías dudas)
te marqué un par de cosas para que tengas en cuenta pero no hace falta que actualices el PR.
Otra cosa que te recomiendo en general es que uses algún formateador tipo prettier si podés, para que quede un poco más organizado el código y sea más fácil de leer. Si usás visual studio code deberías poder instalar la extensión directamente.
| <input type={props.tipo} | ||
| value={value} | ||
| onChange={e=>{setValue(e.target.value)}} | ||
| type={props.isPassword?'password':' '} |
There was a problem hiding this comment.
es algo menor, pero fijate que acá la prop type={props.tipo} nunca se va a aplicar porque se sobreescribe con el type={props.isPassword?'password':' '} de más abajo
| <ValidationInput | ||
| tipo='Nombre' | ||
| isPassword = {false} | ||
| validation = {(value) => value === 'Ana Rodríguez'} | ||
| /> | ||
|
|
||
| <ValidationInput | ||
| tipo='Email' | ||
| isPassword = {false} | ||
| validation = {(value) => !value.match(" ") && value.match(/^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/) } | ||
| /> | ||
|
|
||
| <ValidationInput | ||
| tipo='Password' | ||
| isPassword = {true} | ||
| validation = {(value) => value.length >=8 } | ||
| /> | ||
|
|
There was a problem hiding this comment.
solo para tenerlo en cuenta, si una prop es false, en general no se pasa directamente, y si querés pasar una propiedad como true podés solamente escribir el nombre, ejemplos con tus inputs:
<ValidationInput
tipo="Nombre"
validation={(value) => value === 'Ana Rodríguez'}
/>
<ValidationInput
tipo="Password"
isPassword
validation={(value) => value.length >=8}
/>
No description provided.