Skip to content

Conversation

@matiasperalta1
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 22, 2025 19:25
@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 agrega un nuevo módulo sale_order_type_project que permite asociar un proyecto a un tipo de orden de venta. El módulo es un glue module (auto_install) que conecta funcionalidad entre sale_order_type y sale_project.

  • Agrega un campo project_id al modelo sale.order.type para definir un proyecto predeterminado
  • Extiende la vista de formulario de tipo de orden para mostrar el nuevo campo
  • Establece las dependencias necesarias entre los módulos base

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
sale_order_type_project/__manifest__.py Define el módulo con versión 18.0.1.0.0, declara dependencias de sale_project y sale_order_type, y configura como auto_install
sale_order_type_project/__init__.py Importa el paquete models con el encabezado de copyright estándar
sale_order_type_project/models/__init__.py Importa el modelo sale_order_type
sale_order_type_project/models/sale_order_type.py Extiende sale.order.type agregando el campo Many2one project_id con dominio para proyectos facturables
sale_order_type_project/views/sale_order_type_view.xml Hereda la vista de formulario de tipo de orden para mostrar el campo project_id después de sequence_id con grupos de seguridad apropiados

comodel_name="project.project",
domain=[("allow_billable", "=", True)],
string="Project",
help="Select to define the analytics account",
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

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

Falta especificar el parámetro ondelete en el campo Many2one. Este parámetro define qué ocurre con los registros de sale.order.type cuando se elimina el proyecto relacionado. Se recomienda usar ondelete="restrict" para evitar eliminar proyectos que están siendo usados en tipos de orden, o ondelete="set null" si se prefiere simplemente limpiar la referencia.

Suggested change
help="Select to define the analytics account",
help="Select to define the analytics account",
ondelete="restrict",

Copilot uses AI. Check for mistakes.
Comment on lines +13 to +14
help="Select to define the analytics account",
)
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

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

El texto de ayuda menciona "Select to define the analytics account", pero el módulo actualmente solo agrega el campo project_id al tipo de orden sin implementar ninguna lógica para propagar este valor a las órdenes de venta o sus líneas. Para que el módulo sea funcional, debería incluir:

  1. Un modelo sale.order que herede y propague el project_id desde type_id.project_id a las líneas de orden (donde se utiliza realmente en el módulo sale_project).
  2. Alternativamente, actualizar el texto de ayuda para reflejar que este campo es solo informativo y no se propaga automáticamente.

Copilot generated this review using guidance from repository custom instructions.
##############################################################################
{
"name": "Sale Order Type Project",
"version": "18.0.1.0.0",
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

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

La versión del módulo está configurada como "18.0.1.0.0", pero según el contexto del repositorio (rama 19.0 mencionada en las guías), debería ser "19.0.1.0.0". Es importante que la versión del manifest coincida con la versión de Odoo objetivo.

Copilot generated this review using guidance from repository custom instructions.
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.

2 participants