Skip to content

[19.0][MIG] l10n_es_atc: Migration to 19.0 - #5023

Open
antonyht27 wants to merge 28 commits into
OCA:19.0from
antonyht27:19.0-mig-l10n-es-atc
Open

[19.0][MIG] l10n_es_atc: Migration to 19.0#5023
antonyht27 wants to merge 28 commits into
OCA:19.0from
antonyht27:19.0-mig-l10n-es-atc

Conversation

@antonyht27

Copy link
Copy Markdown
Contributor

No description provided.

Nicolás Ramos and others added 27 commits June 2, 2026 17:31
Currently translated at 100.0% (3 of 3 strings)

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_atc/es/
It should be installed on demand, as having AEAT doesn't mean we should
install ATC.
…eclarations from 4xx models for ATC

[IMP] l10n_es_atc_mod420: Generate XML file for declaration
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_atc/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_atc/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_atc/
Check the headers to verify whether a new file exists before downloading it
Currently translated at 77.3% (65 of 84 strings)

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_atc/es/
Currently translated at 79.7% (67 of 84 strings)

Translation: l10n-spain-18.0/l10n-spain-18.0-l10n_es_atc
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_atc/es/
@antonyht27 antonyht27 mentioned this pull request Jun 2, 2026
40 tasks
@HaraldPanten

Copy link
Copy Markdown
Contributor

/ocabot migration l10n_es_atc

Hola @antonyht27 puedes revisar los commits? Tienes 2 commits que hacen referencia a los cambios del pre-commit. Habría que juntarlos todos en el mismo.

Gracias por la aportación.

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Jun 3, 2026
@antonyht27
antonyht27 force-pushed the 19.0-mig-l10n-es-atc branch from 9a1dcef to c530fe1 Compare June 3, 2026 08:16
@antonyht27

Copy link
Copy Markdown
Contributor Author

/ocabot migration l10n_es_atc

Hola @antonyht27 puedes revisar los commits? Tienes 2 commits que hacen referencia a los cambios del pre-commit. Habría que juntarlos todos en el mismo.

Gracias por la aportación.

Listo, combiné las correcciones automáticas previas a la confirmación duplicadas en una sola.

@jelenapoblet

Copy link
Copy Markdown

@ajfebres @syci podemos revisar esto ?

@Michelle-RiveroM

Copy link
Copy Markdown

Se ha podido revisar esto para que sea aprobado ? @jelenapoblet

@jelenapoblet

Copy link
Copy Markdown

@Michelle-RiveroM , en estos días lo estamos revisando, habría que mirar porque fallan los tests.
ping @antonyht27

@antonyht27

Copy link
Copy Markdown
Contributor Author

Hola @jelenapoblet,

He comprobado los checks y los tests en rojo es Codecov por cobertura del patch/proyecto.

Este módulo viene desde 16.0 sin carpeta tests, por eso la migración a 19.0 queda con baja cobertura en l10n_es_atc_report.py.

¿Preferís que añada ahora una carpeta tests básica para cubrir la lógica migrada y mejorar el Codecov, o en este caso se puede considerar el fallo de cobertura como no bloqueante para la migración?

@ajfebres

ajfebres commented Jun 9, 2026

Copy link
Copy Markdown

Code Review. LGTM!

@antonyht27 Aunque ciertamente viene sin tests desde versiones anteriores, seria preciso aprovechar la ocasión para crear tests básicos para cubrir la lógica.

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.