Open zapaiamarce opened 2 years ago
Buenas marce! Genial, me voy a poner a corregir esas cositas y mejorarla con el tiempo para que quede algo bueno, muchas graciass!!
On Fri, 24 Jun 2022, 11:27 am Marcelo Zapaia, @.***> wrote:
A nivel código creo que está correcto. Obviamente hay muchos patrones y cosa fina que puede mejorar, pero son cosas que vas a ir aprendiendo y es necesario que vayas mejorando este proyecto cada vez que aprendas algo nuevo. Este proyecto es la síntesis de muchas cosas que aprendiste y está bueno que las personas que entren a ver tu proyecto y tu código vean algo copado.
Te dejo algunas cositas de la UI para que tengas en cuenta pero como te decía, anda mejorándola cada vez que puedas!
[image: image] https://user-images.githubusercontent.com/1208547/175555485-2227c326-8b30-4dea-84f5-1d2dd581f31a.png esto no es necesario, directamente que te rediriga a login
[image: image] https://user-images.githubusercontent.com/1208547/175555572-d7b13f5a-d6b3-4dee-a78d-96905579a7ee.png
nombre de usuario para que? la edad para que??
[image: image] https://user-images.githubusercontent.com/1208547/175555982-b6dc1b39-be49-4f41-8763-7886b04e8c6c.png Acá falta el nombre del producto
— Reply to this email directly, view it on GitHub https://github.com/Alvax10/dwf-m10/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/APKSAAW7ZVMCPHEHNQ4MNUTVQXAU3ANCNFSM5ZYA4TOQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>
A nivel código creo que está correcto. Obviamente hay muchos patrones y cosa fina que puede mejorar, pero son cosas que vas a ir aprendiendo y es necesario que vayas mejorando este proyecto cada vez que aprendas algo nuevo. Este proyecto es la síntesis de muchas cosas que aprendiste y está bueno que las personas que entren a ver tu proyecto y tu código vean algo copado.
Te dejo algunas cositas de la UI para que tengas en cuenta pero como te decía, anda mejorándola cada vez que puedas!
esto no es necesario, directamente que te rediriga a login
nombre de usuario para que? la edad para que??
Acá falta el nombre del producto