Arquisoft / dede_es5a

ArquiSocks: la mayor variedad de calcetines. Disponible tanto en Heroku como en Azure: https://dede-es5a.herokuapp.com/ http://20.228.137.74:3000/
2 stars 0 forks source link

Métodos de persistencia añadidos #181

Closed AlonsoA-hub closed 2 years ago

AlonsoA-hub commented 2 years ago

Añadidos los cambios para el issue #173. En el router de productos:

En el router de usuarios:

En el server.ts:

AlonsoA-hub commented 2 years ago

Necesitaría lo siguiente🥺, no es excatamente igual que el producto. Ya que en el carrito pueden estar productos iguales con tallas distintas. image

Listo, ahora ya debería funcionar con ese formato

SofiaYYQ commented 2 years ago

No puedo probarlas por thunder client pq el req.session.cart no se inicializa hasta que el usuario inicie la sesión 😢. Como tal tenemos ahora, un usuario sin iniciar sesión no podrá meter cosas al carrito, no? Igual es mejor inicializar el carrito cuando se va a meter el primer producto 🤔 .

En la imagen estoy intentando push un objeto al carrito, pero está undefined image

AlonsoA-hub commented 2 years ago

No puedo probarlas por thunder client pq el req.session.cart no se inicializa hasta que el usuario inicie la sesión 😢. Como tal tenemos ahora, un usuario sin iniciar sesión no podrá meter cosas al carrito, no? Igual es mejor inicializar el carrito cuando se va a meter el primer producto 🤔 .

En la imagen estoy intentando push un objeto al carrito, pero está undefined image

Es verdad, ahora ya debería estar. En principio si no hay un carrito inicializado se inicializa en la ruta "/products/addCart"

SofiaYYQ commented 2 years ago

No puedo probarlas por thunder client pq el req.session.cart no se inicializa hasta que el usuario inicie la sesión 😢. Como tal tenemos ahora, un usuario sin iniciar sesión no podrá meter cosas al carrito, no? Igual es mejor inicializar el carrito cuando se va a meter el primer producto 🤔 . En la imagen estoy intentando push un objeto al carrito, pero está undefined image

Es verdad, ahora ya debería estar. En principio si no hay un carrito inicializado se inicializa en la ruta "/products/addCart"

Vaa, voy a probarlo ahora

SofiaYYQ commented 2 years ago

Aquí me imagino que es un post en vez de get: image

No me deja probar get /users/session, puede que sea porque no estoy en un navegador?? image

En updateCart y deleteCart, además de mirar si coinciden el id, también el size, sino se nos borran todos los productos q tenga el determinado id aunque tengan distintas tallas. image

AlonsoA-hub commented 2 years ago

Aquí me imagino que es un post en vez de get: image

No me deja probar get /users/session, puede que sea porque no estoy en un navegador?? image

En updateCart y deleteCart, además de mirar si coinciden el id, también el size, sino se nos borran todos los productos q tenga el determinado id aunque tengan distintas tallas. image

Cambié el primer método a POST y cambié también los otros métodos que me dijiste para que miren también el size. Sobre el error no se que puede ser, yo acabo de probar con el thunder client y va bien

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

AlonsoA-hub commented 2 years ago

Perfe, pues hago merge