Skip to content

[FIX] l10n_uy_ux: Extend unlink method to prevent deletion of l10n_uy_edi_error#379

Closed
jue-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-60997-jue
Closed

[FIX] l10n_uy_ux: Extend unlink method to prevent deletion of l10n_uy_edi_error#379
jue-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-60997-jue

Conversation

@jue-adhoc
Copy link
Copy Markdown
Contributor

@jue-adhoc jue-adhoc commented Dec 5, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 5, 2025 17:06
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

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 implementa una extensión del método unlink en el modelo l10n_uy_edi.document para prevenir la eliminación automática de documentos EDI que contienen errores. El módulo oficial l10n_uy_edi elimina estos documentos después de action_post, pero este cambio preserva los documentos con errores para que sean visibles en las facturas posteadas.

Cambios clave:

  • Reordenamiento de imports siguiendo PEP8 (imports de stdlib antes de imports de Odoo)
  • Nuevo método unlink() que filtra documentos con errores y solo elimina aquellos sin errores

Comment thread l10n_uy_ux/models/l10n_uy_edi_document.py Outdated
Comment thread l10n_uy_ux/models/l10n_uy_edi_document.py
Comment thread l10n_uy_ux/models/l10n_uy_edi_document.py
Comment thread l10n_uy_ux/models/l10n_uy_edi_document.py
@jue-adhoc jue-adhoc changed the title [FIX] l10n_uy_ux: Extend unlink method to prevent deletion of l10n_uy… [FIX] l10n_uy_ux: Extend unlink method to prevent deletion of l10n_uy_edi_error Dec 11, 2025
@hav-adhoc
Copy link
Copy Markdown
Contributor

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Dec 16, 2025
…_edi_error

closes #379

Signed-off-by: Victoria Halac - hav (#l10n) <hav@adhoc.inc>
@roboadhoc roboadhoc closed this Dec 16, 2025
@roboadhoc roboadhoc deleted the 18.0-t-60997-jue branch December 16, 2025 12:59
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