Skip to content

[REN] l10n_uy_reports_extended: rename to l10n_uy_reports_ux#365

Closed
zaoral wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-t-21838-v2-kz
Closed

[REN] l10n_uy_reports_extended: rename to l10n_uy_reports_ux#365
zaoral wants to merge 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-t-21838-v2-kz

Conversation

@zaoral
Copy link
Copy Markdown
Contributor

@zaoral zaoral commented Oct 14, 2025

To follow defined standard

Copilot AI review requested due to automatic review settings October 14, 2025 19:24
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

@zaoral zaoral changed the title [REN] l10n_uy_reports_extended; rename to l10n_uy_reports_ux [REN] l10n_uy_reports_extended: rename to l10n_uy_reports_ux Oct 14, 2025
@zaoral zaoral force-pushed the 19.0-t-21838-v2-kz branch from 12e5e54 to b5bafec Compare October 14, 2025 19:26
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

Rename of the module from l10n_uy_reports_extended to l10n_uy_reports_ux to align with a naming standard, updating template references, assets, security access IDs, and documentation accordingly.

  • Renamed XML QWeb template names, asset bundle path, security access record IDs, and manifest module name.
  • Removed old i18n resources under the previous module name and added new POT/PO translation files for the new module.
  • Updated README heading to reflect the new module name.

Reviewed Changes

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

Show a summary per file
File Description
l10n_uy_reports_ux/static/src/components/uruguayan_report/filters.xml Updated QWeb template and t-call names to new module namespace.
l10n_uy_reports_ux/static/src/components/uruguayan_report/filter_tax_type.xml Renamed template t-name to new namespace.
l10n_uy_reports_ux/security/ir.model.access.csv Adjusted access record and model external IDs to new module prefix.
l10n_uy_reports_ux/models/l10n_uy_vat_book.py Updated template reference string in configuration dict.
l10n_uy_reports_ux/i18n/l10n_uy_reports.pot Added new POT file with original English strings (contains spelling/grammar issues).
l10n_uy_reports_ux/i18n/es.po Added Spanish translations (contains several spelling/grammar issues).
l10n_uy_reports_ux/manifest.py Changed module display name and asset bundle path.
l10n_uy_reports_ux/README.rst Updated title to include new module name.
l10n_uy_reports_extended/i18n/l10n_uy_reports.pot Removed old POT file tied to previous module name.
l10n_uy_reports_extended/i18n/es.po Removed old Spanish PO file tied to previous module name.
Comments suppressed due to low confidence (1)

l10n_uy_reports_ux/i18n/es.po:1

  • Number and gender agreement: singular 'Asiento' with plural 'Abiertos'. Suggested minimal correction: Asiento Contable Abierto
# Translation of Odoo Server.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread l10n_uy_reports_ux/i18n/l10n_uy_reports.pot Outdated
Comment thread l10n_uy_reports_ux/i18n/l10n_uy_reports.pot Outdated
Comment thread l10n_uy_reports_ux/i18n/es.po Outdated
Comment thread l10n_uy_reports_ux/i18n/es.po Outdated
Comment thread l10n_uy_reports_ux/i18n/es.po Outdated
@zaoral zaoral force-pushed the 19.0-t-21838-v2-kz branch from b5bafec to b54accd Compare October 14, 2025 20:00
@zaoral zaoral force-pushed the 19.0-t-21838-v2-kz branch 2 times, most recently from b848031 to dd16926 Compare November 28, 2025 17:27
@zaoral zaoral force-pushed the 19.0-t-21838-v2-kz branch 2 times, most recently from 8d01816 to b5789aa Compare January 5, 2026 20:05
Rename of the module from l10n_uy_reports_extended to l10n_uy_reports_ux
to align with a naming standard, updating template references, assets,
security access IDs, and documentation accordingly.

Migration to old adhoc module l10n_uy_reports to version 19.0
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

Copilot reviewed 15 out of 22 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (3)

l10n_uy_reports_ux/manifest.py:4

  • El cambio de versión de 18.0.1.1.0 a 19.0.1.0.0 junto con el renombrado del módulo requiere un script de migración. El renombrado del módulo de l10n_uy_reports_extended a l10n_uy_reports_ux implica cambios en:
  • Referencias en ir.model.data (XML IDs)
  • Referencias en el archivo CSV de seguridad (ir.model.access.csv)
  • Assets del backend en el manifest

Se necesita un script de migración en migrations/19.0.1.0.0/pre_rename_module.py que actualice todas las referencias del módulo antiguo al nuevo antes de que Odoo intente cargar el módulo renombrado. Este script debe:

  1. Actualizar registros en ir.model.data cambiando el campo module de l10n_uy_reports_extended a l10n_uy_reports_ux
  2. Actualizar registros en ir.model.access que referencien el módulo antiguo
  3. Ser idempotente para permitir múltiples ejecuciones seguras
    l10n_uy_reports_ux/wizards/form_report_wiz.py:185
  • La traducción del mensaje de error tiene una inconsistencia terminológica. Cambias "numero de vat" a "número de ID Fiscal", pero el código sigue usando el campo self.company_id.vat. Para mantener consistencia con el código subyacente, sería más apropiado: "Configure un número de VAT para su compañía" (con tilde en número pero manteniendo VAT que es el término técnico usado en el campo).
    l10n_uy_reports_ux/report/account_uy_vat_line_views.xml:78
  • El menú account_reports_uy_statements_menu se movió de data/account_financial_report_data.xml (líneas eliminadas) a report/account_uy_vat_line_views.xml (líneas 77-78 nuevas). Esta reestructuración es correcta, pero al ser un cambio en la ubicación de un registro XML con noupdate implícito, puede causar duplicados si el módulo ya estaba instalado.

Verificar si se necesita un script de migración post que elimine el registro duplicado del XML ID account_reports_uy_statements_menu de la ubicación anterior, o asegurar que ambos archivos XML usen el mismo XML ID para que Odoo actualice en lugar de duplicar.

@pablohmontenegro
Copy link
Copy Markdown
Contributor

@roboadhoc r+

@roboadhoc roboadhoc closed this in 1e1a144 Jan 6, 2026
@roboadhoc roboadhoc added the 18.1 label Jan 6, 2026
@roboadhoc roboadhoc deleted the 19.0-t-21838-v2-kz branch January 6, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants