search

Nahuel0 / Proyecto-Final-Misiotronica

Este proyecto fue inspirado por la facultad FCEQYN de Argentina Programa 4.0 - Realizado bajo React
0 stars 0 forks source link

Routeado definido pero no utilizado #1

Open maxpipoka opened 1 year ago

maxpipoka commented 1 year ago

Hola Nahuel, te consulto. Veo que tenes definido el routeo en el App.js pero no lo estas utilizando. Tu navbar esta apuntando de manera estática a elementos html mediante el 'name'. Falta arreglar aun eso?

https://github.com/Nahuel0/Proyecto-Final-Misiotronica/blob/02bb64ac90d516cff0796dabcb9200d58d0ea0dc/src/App.js#L14C13-L14C13

Nahuel0 commented 1 year ago

que tal martin, si, no me había dado cuenta. Desde las nav, debería usar la url http:loca.../ventas ? y asi verdad ? porque esta el ruteo pero por navegacion promia digamos si pones "http://localhost:3000/tienda" te deberia aparecer la tienda, asi con las demas, me falta cambiarlo en la nav. gracias!

Una pregunta, es realmente necesario el tema de REDUX? Intento implementarlo en mi app, pero se me complica mucho insertarlo ya que tenia todo bastante realizado (Solo me falta el carrito pero pensé que no era un requerimiento!)

El lun, 31 jul 2023 a las 23:31, Martin Cardozo @.***>) escribió:

Hola Nahuel, te consulto. Veo que tenes definido el routeo en el App.js pero no lo estas utilizando. Tu navbar esta apuntando de manera estática a elementos html mediante el 'name'. Falta arreglar aun eso?

https://github.com/Nahuel0/Proyecto-Final-Misiotronica/blob/02bb64ac90d516cff0796dabcb9200d58d0ea0dc/src/App.js#L14C13-L14C13

— Reply to this email directly, view it on GitHub https://github.com/Nahuel0/Proyecto-Final-Misiotronica/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARAN7WHTBFZHAX555ARAULLXTBTALANCNFSM6AAAAAA27CP43Y . You are receiving this because you are subscribed to this thread.Message ID: @.***>

maxpipoka commented 1 year ago

Hola Nahuel, claro. Tu navbar debería apuntar a las rutas creadas en el router. Además de que podes controlar el renderizado mediante el router también. No haría falta mostrar todo. Si implementas router tenés que usarlo, sino no tiene sentido. En los requerimientos se pide así que tenés que hacerlo. Redux es optativo. Si se pide el manejo de estados y eventos. Pero lo podrías hacer con useState.

Avisa cuando arregles lo del enrutado comentando esta issue nuevamente.

Nahuel0 commented 1 year ago

Martin ahi corregi el tema de la NAV y tambien de contacto que estaba con anclas tambien. Fuero corregidos. Cualquier otra cosa me avisas. Ahora voy a crear otra rama para ver si puedo hacer la funcionalidad del carrito. Si veo que no llego no lo subire. Pero voy a intentarlo. Repito, si debo cambiar otra cosa me avisas. Sldos!

maxpipoka commented 1 year ago

Hola Nahuel, dale, después reviso. Si esta todo ok cierro este issue. Si surge otra cosa creo otro nuevo. Sobre la rama para el carrito... si nno llega a salir, igual subilo, asi que da el intento igual registrado, y te sirve para seguirlo trabajando en otro momentno.

Saludos y ánimos!

Nahuel0 commented 1 year ago

Gracias Martin, ahi agregue la nueva rama con el proceso del carrito en curso!, Me podria confirmar si esta todo en orden ? osea si lo ve bien y esta bien realizado la subida :)

maxpipoka commented 1 year ago

Si figura subida la segunda rama. Tambien figura un comit en la rama principal con modificaciones, fijate.

Nahuel0 commented 1 year ago

Martin buenas tardes, le comento que ya modifique y subi y uni la rama de carrito con la rama main, ya todo se deberia ver en main. Pude hacer la funcionalidad de carrito. No lo hice con redux porque se me complico demasiado. Ya creo que todo estaria finalizado!

Sldos!