Alvax10 / Lost_pets_react

0 stars 0 forks source link

Review #11

Open alexismunoz1 opened 2 years ago

alexismunoz1 commented 2 years ago

Alva ahora si, el desafío cumple con las condiciones de ser aprobado!

Alvax10 commented 2 years ago

Muchas gracias Alex!! Mañana me pongo a corregir esos errores y me pongo las pilas con el módulo 9! Muchisimas graciass!

El lun., 28 de marzo de 2022 11:32 p. m., Alexis Muñoz < @.***> escribió:

Alva ahora si, el desafío cumple con las condiciones de ser aprobado!

-

Hay algunos detalles por corregir:

Cuando editas una mascota y no se ingresa el nombre de la mascota, setea el mismo con un string vasío, podrías hacer que si el campo está vacío, el valor sea el nombre actual de la mascota: [image: Captura de pantalla de 2022-03-28 23-03-03] https://user-images.githubusercontent.com/77214476/160519254-fd5f5747-523e-48a2-bf24-7b053f381a79.png

Estaría bueno que en mis datos aparezcan mis datos, como el mail y el nombre: [image: https://user-images.githubusercontent.com/77214476/160142336-8cb93c01-cc29-4271-87d7-7d150cc02f27.png] https://user-images.githubusercontent.com/77214476/160142336-8cb93c01-cc29-4271-87d7-7d150cc02f27.png

Se dice que es una buena práctica pasar 2 o 3 argumentos como mucho a una función, lo recomendable en este caso sería pasar un parámetro en forma de objeto: [image: https://user-images.githubusercontent.com/77214476/160142574-910d9f0f-1f86-4af5-a6ea-b84c8d9883ec.png] https://user-images.githubusercontent.com/77214476/160142574-910d9f0f-1f86-4af5-a6ea-b84c8d9883ec.png

Lo demás está bien y más allá de esos detalles son cosas que se van aprendiendo a medida que se labura con la herramienta (react), éxitos en lo que sigue!!

— Reply to this email directly, view it on GitHub https://github.com/Alvax10/dwf-m8-desafio/issues/11, or unsubscribe https://github.com/notifications/unsubscribe-auth/APKSAAWGLOMEL7VBNVE3ZRLVCJTT5ANCNFSM5R46CP3A . You are receiving this because you are subscribed to this thread.Message ID: @.***>