Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cart behavior fix #147

Closed
wants to merge 1 commit into from
Closed

Cart behavior fix #147

wants to merge 1 commit into from

Conversation

Bastian1998
Copy link
Contributor

@Bastian1998 Bastian1998 commented Dec 17, 2021

Removed unnecessary behavior in cart that cart-item already implements( get cart item by cart id)

-Cart-Item has endpoint to get all cart-items from a cart that matches given id
image
image

@Bastian1998 Bastian1998 self-assigned this Dec 17, 2021
@Bastian1998 Bastian1998 linked an issue Dec 17, 2021 that may be closed by this pull request
@Bastian1998
Copy link
Contributor Author

@ChachoPacho aqui esta el pr para solo remover el comportamiento que ya tenia el cart-item, no toque el problema que menciona ale con el purchased_at no se si @Marcosl14 sigue trabajando con eso. No deberia generar ningun conflicto ya que no dependia de informacion del cart sino que llamaba al servicio de cart-items desde cart

@Alonso-Pablo
Copy link
Member

Si eso es lo que necesita el front-end para renderizar las publicaciones/productos que tiene en el carrito estaría faltando datos. El cartId no serviría de nada y es mejor no pasarlo porque no es necesario que el usuario o el front sepa que id tiene el carrito actual. Luego esta "itemId" que tendría que ser reemplazado por "item" con todos los datos ("title", "photos", "price", etc) para que el front-end tenga datos y renderizarlo en la página, ya que el id del ítem poco serviría.

Puede que me equivoque, cualquier cosa decime si el fin no es el que dije. 🤔

@Bastian1998
Copy link
Contributor Author

@Alonso-Pablo puedo hacer el paginado del cart y mostrar los items que lleva dentro, segun lo que vimos en clase solo se necesita el enpoint GET/ de cart y no uno que lo busque por id .

@alezvi alezvi closed this Jan 7, 2022
@ghnoob ghnoob deleted the cart-get-cart-items-fix branch January 12, 2022 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cart unnecessary behavior fix
3 participants