PR de corrección (NO MERGEAR) #9
Conversation
| function validation(value) { | ||
| return value.length >= 8 | ||
| } |
There was a problem hiding this comment.
como te había comentado en slack, la idea de esta función es que sea una prop más.
fijate si podés renderizar esto en index-mio.js y lograr que se use esa función en vez de esta:
ReactDOM.render(
<ValidationInput validation={(value) => value.length >= 8} />,
document.getElementById("react-app")
)
| const [items, setItems] = React.useState(props.items) | ||
| return( | ||
| <div> | ||
| {Object.keys(items).map((key) => { | ||
| return ( | ||
|
|
||
|
|
||
| <ControlledCheckbox name={key} value={items[key]} onChange={ | ||
| (value) => {setItems(!value)} | ||
| } /> | ||
|
|
||
| ) | ||
| })} | ||
| </div> | ||
| } |
There was a problem hiding this comment.
efectivamente, este componente no hace lo que debería, pero no te preocupes demasiado por eso, este ejercicio en particular era más complicado de lo que esperaba, y la mayoría de la gente no lo pudo resolver solo con lo visto en la clase 1. te recomendaría arrancar la clase 2 y ver la corrección que hago ahí.
el problema que tenés acá es que estás tratando de cambiar items de un objeto a un booleano. hay varias formas de resolver esto, pero la que yo en general recomiendo usar es crear un objeto nuevo con spread syntax y setear la key que estás cambiando ahí mismo:
(value) => {
setItems({...items, [key]: !value})
}
| <input | ||
| type="checkbox" | ||
| name={props.name} | ||
| value={props.onChange || props.value} |
There was a problem hiding this comment.
ojo que acá en value tiene que ir props.value, no estoy muy seguro porque pasás props.onChange también. igual entiendo que al no haberte funcionado el otro componente no pudiste testear bien este.
No description provided.