Skip to content

[IMP]website_sale_ux: remove custom add to cart behaviour#457

Closed
JrAdhoc wants to merge 1 commit into
ingadhoc:18.0from
adhoc-dev:18.0-t-68182-jr
Closed

[IMP]website_sale_ux: remove custom add to cart behaviour#457
JrAdhoc wants to merge 1 commit into
ingadhoc:18.0from
adhoc-dev:18.0-t-68182-jr

Conversation

@JrAdhoc

@JrAdhoc JrAdhoc commented May 21, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 21, 2026 15:31
@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR simplifica el módulo website_sale_ux eliminando la personalización del “Add to Cart” en el listado de productos, volviendo al comportamiento estándar de website_sale y retirando la traducción asociada.

Changes:

  • Se elimina la plantilla QWeb heredada products_add_to_cart que alteraba el “quick add” según cantidad de variantes.
  • Se quita la cadena traducible “Shopping cart (Select attributes)” del es.po del módulo.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
website_sale_ux/views/product_template_views.xml Elimina la herencia de website_sale.products_add_to_cart que introducía comportamiento/botón custom.
website_sale_ux/i18n/es.po Elimina la traducción asociada a la vista removida.

@augusto-weiss

Copy link
Copy Markdown
Contributor

@roboadhoc r+ bump

roboadhoc pushed a commit that referenced this pull request May 26, 2026
closes #457

Signed-off-by: Augusto Weiss <awe@adhoc.com.ar>
roboadhoc added a commit that referenced this pull request May 26, 2026
@roboadhoc roboadhoc closed this May 26, 2026
@roboadhoc roboadhoc deleted the 18.0-t-68182-jr branch May 26, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants