Skip to content

[FIX] l10n_ar_sale: fix sale order report for Argentina#280

Closed
pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-110727-pam
Closed

[FIX] l10n_ar_sale: fix sale order report for Argentina#280
pablohmontenegro wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-110727-pam

Conversation

@pablohmontenegro
Copy link
Copy Markdown
Contributor

Ticket: 110727

Copilot AI review requested due to automatic review settings February 13, 2026 13:40
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 corrige el renderizado del reporte de pedido de venta para compañías de Argentina en l10n_ar_sale, ajustando cómo se selecciona el template QWeb a utilizar según el país de la compañía.

Changes:

  • Condiciona el t-call del template base para que no se use cuando la compañía es AR.
  • Añade un t-call alternativo a l10n_ar_sale.report_saleorder_document cuando la compañía es AR.
  • Incrementa la versión del módulo.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
l10n_ar_sale/views/sale_report_templates.xml Cambia la lógica de selección del template de reporte para usar el layout AR solo cuando corresponde.
l10n_ar_sale/manifest.py Bump de versión del módulo por cambio en vista QWeb.

Comment thread l10n_ar_sale/views/sale_report_templates.xml
@jue-adhoc
Copy link
Copy Markdown
Contributor

@roboadhoc nobump

@jue-adhoc
Copy link
Copy Markdown
Contributor

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Feb 13, 2026
closes #280

Ticket: 110727
Signed-off-by: Julia Elizondo - jue (#l10n) <jue@adhoc.com.ar>
@roboadhoc roboadhoc closed this Feb 13, 2026
@roboadhoc roboadhoc deleted the 18.0-h-110727-pam branch February 13, 2026 14:21
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