Skip to content

[FIX] l10n_uy_ux: add default legends to invoices lines#404

Closed
jue-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-113287-jue
Closed

[FIX] l10n_uy_ux: add default legends to invoices lines#404
jue-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-113287-jue

Conversation

@jue-adhoc
Copy link
Copy Markdown
Contributor

No description provided.

@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

@jue-adhoc jue-adhoc force-pushed the 18.0-h-113287-jue branch 2 times, most recently from f7cdd5e to a681538 Compare March 19, 2026 16:08
@jue-adhoc jue-adhoc marked this pull request as ready for review March 19, 2026 16:08
Copilot AI review requested due to automatic review settings March 19, 2026 16:08
Copy link
Copy Markdown

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 busca que las leyendas/addendas por defecto (tipo item) se incorporen automáticamente en la descripción (DscItem) de las líneas de factura al generar el CFE en l10n_uy_ux.

Changes:

  • Se incorpora lógica para combinar addendas por defecto con las addendas configuradas en cada línea al construir NomItem/DscItem.
  • Se ajustan algunos filtros/lambdas para mejorar legibilidad.
  • Se añade un override en account.move.line para anular una constraint de Odoo relacionada a addendas.

Reviewed changes

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

File Description
l10n_uy_ux/models/account_move_line.py Override de constraint de addendas en líneas (actualmente se desactiva por completo).
l10n_uy_ux/models/account_move.py Inclusión de addendas por defecto en la construcción de DscItem y ajuste de uso en el popup de “mandatory legend”.
l10n_uy_ux/models/__init__.py Se importa el nuevo archivo de modelo para registrar el override.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread l10n_uy_ux/models/account_move_line.py Outdated
Comment thread l10n_uy_ux/models/account_move.py Outdated
Comment thread l10n_uy_ux/models/account_move.py Outdated
Copy link
Copy Markdown
Contributor

@zaoral zaoral left a comment

Choose a reason for hiding this comment

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

Comment thread l10n_uy_ux/models/account_move.py Outdated
Comment thread l10n_uy_ux/models/account_move.py Outdated
@jue-adhoc jue-adhoc force-pushed the 18.0-h-113287-jue branch 3 times, most recently from c270385 to b7f7033 Compare March 19, 2026 17:55
</list>
</field>
<span>Preview: </span>
<div style="margin-top: 8px; margin-bottom: 8px;"><strong>Preview:</strong></div>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ocultamos la vista acá porque sólo se muestran las leyendas que no son de tipo "item" y puede generar confusiones. Para ver qué adendas o leyendas hay, se pueden usar los botones de abajo. Además, con el botón "Get defaults" escribimos los campos en la pestaña "Otra info" y en la línea de factura

Copy link
Copy Markdown
Contributor

@zaoral zaoral left a comment

Choose a reason for hiding this comment

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

Thank you for the PR @jue-adhoc
@roboadhoc bump
@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Mar 19, 2026
closes #404

Signed-off-by: Katherine Zaoral - kz (#l10n) <kz@adhoc.com.ar>
roboadhoc added a commit that referenced this pull request Mar 19, 2026
@roboadhoc roboadhoc closed this Mar 19, 2026
@roboadhoc roboadhoc deleted the 18.0-h-113287-jue branch March 19, 2026 20:35
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