Skip to content

Comments

[IMP]website_sale_order_validity: show button when validity is not set#440

Closed
JrAdhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-62332-jr
Closed

[IMP]website_sale_order_validity: show button when validity is not set#440
JrAdhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-62332-jr

Conversation

@JrAdhoc
Copy link
Contributor

@JrAdhoc JrAdhoc commented Jan 7, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 7, 2026 16:04
@roboadhoc
Copy link

Pull request status dashboard

Copy link

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 mejora la lógica de visualización de botones de navegación en el checkout del sitio web para órdenes de venta con validez. Anteriormente, los botones solo se mostraban cuando había una fecha de validez establecida y válida; ahora también se muestran cuando no hay fecha de validez configurada.

  • Cambia la condición en la plantilla XML para mostrar botones de navegación cuando validity_date no está establecida O cuando está vigente
  • Incrementa la versión del módulo de 18.0.1.0.0 a 18.0.1.1.0

Reviewed changes

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

File Description
website_sale_order_validity/views/templates.xml Modifica la condición t-if para mostrar botones de navegación cuando no hay fecha de validez o cuando la fecha es válida
website_sale_order_validity/manifest.py Incrementa la versión del módulo a 18.0.1.1.0 para reflejar el cambio en la vista

He revisado cuidadosamente este PR y la lógica del cambio es correcta y consistente con el resto del código. El cambio permite que los botones de navegación se muestren tanto cuando no hay fecha de validez configurada como cuando la fecha es válida, lo cual es el comportamiento esperado. La versión se incrementó apropiadamente según las guías de codificación para cambios en vistas XML. No he encontrado problemas que requieran comentarios.

@cem-adhoc
Copy link
Contributor

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Jan 7, 2026
closes #440

Signed-off-by: Manuela Cepeda <cem@adhoc.com.ar>
@roboadhoc roboadhoc closed this Jan 7, 2026
@roboadhoc roboadhoc deleted the 18.0-t-62332-jr branch January 7, 2026 17:19
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.

3 participants